On Fri, Jul 31, 2015 at 10:05 AM, Jason Ekstrand <ja...@jlekstrand.net> wrote: > The following 5 patches contain a few register allocation cleanups and > performance improvements. Chris Wilson noticed that setting up register > sets on i965 calls reralloc an absurd number of times. I did a little > hacking and found out that the initial size for the collision lists is way > too low. This series also contains a patch to avoid setting up registers > more times than needed on platforms where RA is the same for SIMD8 vs > SIMD16. > > The whole series seems to cut about 4 minutes off a piglit run on BYT. It > usually takes around 31 minutes and this time it ran in 27. > > Jason Ekstrand (5): > ra: Refactor ra_set_finalize > ra: Delete the conflict lists in ra_set_finalize > ra: Allocate bigger initial conflict lists > i965/fs: Use dispatch_width instead of reg_width in alloc_reg_sets
The first four are Reviewed-by: Matt Turner <matts...@gmail.com> > i965/fs: Don't do redundant RA setup on IVB+ I'll take your word that the register sets are identical between SIMD8/16 on IVB+ and give you an Acked-by: Matt Turner <matts...@gmail.com> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
