Re: [Mesa-dev] [PATCH 2/3] nir: Fix load_const comparisons for CSE.

2015-02-06 Thread Connor Abbott
Maybe value.f[0] since it's an array and not a pointer? Other than that, Reviewed-by: Connor Abbott On Sat, Feb 7, 2015 at 12:16 AM, Eric Anholt wrote: > We want the size of a float per component, not the size of a whole vec4. > > NIR instructions on i965: > total instructions in shared program

[Mesa-dev] [PATCH 2/3] nir: Fix load_const comparisons for CSE.

2015-02-06 Thread Eric Anholt
We want the size of a float per component, not the size of a whole vec4. NIR instructions on i965: total instructions in shared programs: 1261937 -> 1261929 (-0.00%) instructions in affected programs: 114 -> 106 (-7.02%) Looking at one of these examples (tesseract), it's from vec4 load_consts