[Mesa-dev] [PATCH 02/15] i965: enable component packing for vs and fs

[Timothy Arceri]

On Wed, 2016-07-20 at 12:15 -0700, Kenneth Graunke wrote:
> On Tuesday, July 19, 2016 4:33:14 PM PDT Timothy Arceri wrote:
> > Rather than trying to work out the total number of components
> > used at a location we simply treat all outputs as vec4s.
> > ---
> >  src/mesa/drivers/dri/i965/brw_fs.h           |  1 -
> >  src/mesa/drivers/dri/i965/brw_fs_nir.cpp     | 22 ++++++++++----
> > --------
> >  src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 10 ++--------
> >  src/mesa/drivers/dri/i965/brw_nir.c          |  8 ++++----
> >  4 files changed, 16 insertions(+), 25 deletions(-)
> 
> I ran this through shader-db on Broadwell and got:
> 
> total instructions in shared programs: 9071398 -> 9071272 (-0.00%)
> instructions in affected programs: 2554 -> 2428 (-4.93%)
> helped: 26
> HURT: 0

I wasn't expecting anything to be using it. Nice :)

> 
> Patches 1-3 are
> Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
> 
> Patch 4 is
> Acked-by: Kenneth Graunke <kenneth at whitecape.org>
> 
> Feel free to turn it on for Broadwell and up!
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev