Page MenuHome

Cycles: add shadow path compaction
ClosedPublic

Authored by Brecht Van Lommel (brecht) on Oct 20 2021, 6:48 PM.

Details

Summary

Similar to main path compaction that happens before adding work tiles, this
compacts shadow paths before launching kernels that may add shadow paths.

Only do it when more than 50% of space is wasted.

It's not a clear win in all scenes, some are up to 1.5% slower. Likely caused
by different order of scheduling kernels having an unpredictable performance
impact. Still feels like compaction is just the right thing to avoid cases
where a few shadow paths can hold up a lot of main paths.

Diff Detail

Event Timeline

Brecht Van Lommel (brecht) requested review of this revision.Oct 20 2021, 6:48 PM
Brecht Van Lommel (brecht) created this revision.

From the code side and tests seems all fine, although the performance gain is less on RTX6000:

This revision is now accepted and ready to land.Oct 21 2021, 2:27 PM