Re: [Mesa-dev] V8 Loop unrolling in NIR

2016-12-22 Thread Jason Ekstrand
I have no real opinion on patch 13. Patch 1 is mine so you need to review it. :-) Patch 3/13 needs a fix or it will break clip and cull. We don't need 6/13 or lcssa_phi at all anymore. Pending fixes to the loop unrolling pass to get rid of Vulkan regressions, the series (except 13) is Reviewed

[Mesa-dev] V8 Loop unrolling in NIR

2016-12-21 Thread Timothy Arceri
V8: - bunch of tidy ups mostly suggestions from Jason. - fixed some regressions since V6 caused by V7 handling less trivial loops. Piglit tests sent to cover these cases. New piglit tests: https://patchwork.freedesktop.org/patch/129066/ This series seems to regress some Vulkan CTS tests but