Re: [Mesa-dev] [PATCH 01/15] i965/fs: Remove force_sechalf stack

2013-11-13 Thread Chris Forbes
This patch is Reviewed-by: Chris Forbes This does impact on the dual-simd8 texture support, but that should be fixable similarly. -- Chris On Thu, Nov 14, 2013 at 12:51 PM, Anuj Phogat wrote: > On Tue, Nov 12, 2013 at 5:50 PM, Kenneth Graunke > wrote: >> Only Gen4 color write setup uses the

Re: [Mesa-dev] [PATCH 01/15] i965/fs: Remove force_sechalf stack

2013-11-13 Thread Anuj Phogat
On Tue, Nov 12, 2013 at 5:50 PM, Kenneth Graunke wrote: > Only Gen4 color write setup uses the force_sechalf flag, and it only > sets it on a single instruction. It also already has to get a pointer > to the instruction and manually set the saturate flag, so we may as well > just set force_sechal

[Mesa-dev] [PATCH 01/15] i965/fs: Remove force_sechalf stack

2013-11-12 Thread Kenneth Graunke
Only Gen4 color write setup uses the force_sechalf flag, and it only sets it on a single instruction. It also already has to get a pointer to the instruction and manually set the saturate flag, so we may as well just set force_sechalf the same way and avoid the complexity of a stack. Signed-off-b