Re: [Mesa-dev] [PATCH 13/15] i965/fs: Optimize OR with identical sources into a MOV.

2013-10-29 Thread Matt Turner
On Tue, Oct 29, 2013 at 2:26 PM, Matt Turner wrote: > But hopefully your patch just avoids this problem entirely. I'll > provide an update later. 7 shader helped, 4 shaders hurt by going from my series and your patch, to my series, your patch, and disabling the original SEL-generating function fo

Re: [Mesa-dev] [PATCH 13/15] i965/fs: Optimize OR with identical sources into a MOV.

2013-10-29 Thread Matt Turner
On Tue, Oct 29, 2013 at 12:36 PM, Eric Anholt wrote: > Matt Turner writes: > >> Helps a lot of Steam games. >> >> total instructions in shared programs: 1287432 -> 1287130 (-0.02%) >> instructions in affected programs: 21447 -> 21145 (-1.41%) > > 13-15 are also: > > Reviewed-by: Eric Anholt

Re: [Mesa-dev] [PATCH 13/15] i965/fs: Optimize OR with identical sources into a MOV.

2013-10-29 Thread Eric Anholt
Matt Turner writes: > Helps a lot of Steam games. > > total instructions in shared programs: 1287432 -> 1287130 (-0.02%) > instructions in affected programs: 21447 -> 21145 (-1.41%) 13-15 are also: Reviewed-by: Eric Anholt I'm spending a while thinking about the if->sel passes. Having 2

[Mesa-dev] [PATCH 13/15] i965/fs: Optimize OR with identical sources into a MOV.

2013-10-28 Thread Matt Turner
Helps a lot of Steam games. total instructions in shared programs: 1287432 -> 1287130 (-0.02%) instructions in affected programs: 21447 -> 21145 (-1.41%) --- src/mesa/drivers/dri/i965/brw_fs.cpp | 8 1 file changed, 8 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp