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

Kenneth Graunke kenneth at whitecape.org
Wed Jul 20 19:15:04 UTC 2016


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

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!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160720/546937b3/attachment.sig>


More information about the mesa-dev mailing list