Patches 1 through 4 and 8 are Reviewed-by: Ian Romanick <ian.d.roman...@intel.com>
I don't know the code well enough to give any evaluation of patch 5, and all the others really depend on 5. On 03/11/2014 11:48 PM, Kenneth Graunke wrote: > Hello, > > This series adds support for pull constants in SIMD16 mode. By the end > of the series, shader-db gains 78 extra SIMD16 programs, and the net > code change is -39 lines. In short: > > <anholt> better living through deleting code. > > Most of the work in this series is refactoring the code to change how > information about locations is tracked and how it flows between functions. > Overall, I think it's much nicer at the end (though I'm clearly biased). > > This is available in the 'pull16-v12' branch of ~kwg/mesa. I tested it > for Piglit regressions and shader-db code quality regressions at every > single commit---getting a patch series that didn't regress at any point > was kind of tricky. Lots of obviously correct code would break > horribly...hence the 12 branch revisions. :) > > --Ken > > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
