Page MenuHome

Cycles: remove __UV__ and __INSTANCING__ as kernel options
ClosedPublic

Authored by Brecht Van Lommel (brecht) on Jun 12 2020, 10:48 PM.

Details

Summary

The kernel did not work correctly when these were disabled anyway. The
optimized BVH traversal for the no instances case was also only used on
the CPU, so no longer makes sense to keep.

Ref T73778

Depends on D8010

Diff Detail

Event Timeline

Just to make it clear: those options were purely development-only. There is no performance impact for none of the kernels (even with OpenCL's feature adaptive kernel compilation which never disabled instancing).

This revision is now accepted and ready to land.Jun 17 2020, 2:58 PM