workaround for ICC2016 compiler bug

Author: svenwoop

kernels/builders/bvh_builder_msmblur.h

@@ -131,7 +131,8 @@ namespace embree
           return PrimRefMB (lbounds, tbounds.size(), num_time_segments, geomID, primID);
         }
 
-        __forceinline PrimRefMB operator() (const PrimRefMB& prim, const BBox1f time_range, const LinearSpace3fa& space) const
+        // __noinline is workaround for ICC16 bug under MacOSX
+        __noinline PrimRefMB operator() (const PrimRefMB& prim, const BBox1f time_range, const LinearSpace3fa& space) const
         {
           const unsigned geomID = prim.geomID();
           const unsigned primID = prim.primID();
@@ -146,7 +147,8 @@ namespace embree
           return scene->get<Mesh>(prim.geomID())->linearBounds(prim.primID(), time_range);
         }
 
-        __forceinline LBBox3fa linearBounds(const PrimRefMB& prim, const BBox1f time_range, const LinearSpace3fa& space) const {
+        // __noinline is workaround for ICC16 bug under MacOSX
+        __noinline LBBox3fa linearBounds(const PrimRefMB& prim, const BBox1f time_range, const LinearSpace3fa& space) const {
           return scene->get<Mesh>(prim.geomID())->linearBounds(space, prim.primID(), time_range);
         }
       };

kernels/builders/heuristic_binning_array_unaligned.h

@@ -258,7 +258,7 @@ namespace embree
           }
 
           /*! returns bounds and centroid used for binning */
-          __forceinline void binBoundsAndCenter (const PrimRefMB& ref, BBox3fa& bounds_o, Vec3fa& center_o) const
+          __noinline void binBoundsAndCenter (const PrimRefMB& ref, BBox3fa& bounds_o, Vec3fa& center_o) const // __noinline is workaround for ICC16 bug under MacOSX
           {
             NativeCurves* mesh = scene->get<NativeCurves>(ref.geomID());
             LBBox3fa lbounds = mesh->linearBounds(space,ref.primID(),time_range);
@@ -267,7 +267,7 @@ namespace embree
           }
 
           /*! returns bounds and centroid used for binning */
-          __forceinline void binBoundsAndCenter (const PrimRefMB& ref, LBBox3fa& bounds_o, Vec3fa& center_o) const
+          __noinline void binBoundsAndCenter (const PrimRefMB& ref, LBBox3fa& bounds_o, Vec3fa& center_o) const // __noinline is workaround for ICC16 bug under MacOSX
           {
             NativeCurves* mesh = scene->get<NativeCurves>(ref.geomID());
             LBBox3fa lbounds = mesh->linearBounds(space,ref.primID(),time_range);