Re: [Mesa-dev] Clamp/saturate optimizations v3

2014-08-28 Thread Abdiel Janulgue
On 26.08.2014 15:17, Abdiel Janulgue wrote: > On 23.08.2014 02:57, Ian Romanick wrote: >> Patches 2, 3, 4, 5, 6, 9, 10, 11, 12, 15, and 17 are >> >> Reviewed-by: Ian Romanick >> >> (Additional question below.) >> >> On 08/18/2014 05:17 AM, Abdiel Janulgue wrote: >>> v3 of clamp and saturate opti

Re: [Mesa-dev] Clamp/saturate optimizations v3

2014-08-26 Thread Abdiel Janulgue
On 23.08.2014 02:57, Ian Romanick wrote: > Patches 2, 3, 4, 5, 6, 9, 10, 11, 12, 15, and 17 are > > Reviewed-by: Ian Romanick > > (Additional question below.) > > On 08/18/2014 05:17 AM, Abdiel Janulgue wrote: >> v3 of clamp and saturate optimizations >> >> Changes since v1: >> - Only remove

Re: [Mesa-dev] Clamp/saturate optimizations v3

2014-08-22 Thread Ian Romanick
Patches 2, 3, 4, 5, 6, 9, 10, 11, 12, 15, and 17 are Reviewed-by: Ian Romanick (Additional question below.) On 08/18/2014 05:17 AM, Abdiel Janulgue wrote: > v3 of clamp and saturate optimizations > > Changes since v1: > - Only remove the old try_emit_saturate operations after the new > opti

Re: [Mesa-dev] Clamp/saturate optimizations v3

2014-08-20 Thread Abdiel Janulgue
On 20.08.2014 05:40, Matt Turner wrote: > > Patches 2-4, (5-9 already reviewed), 10, 13-16, (17 already reviewed) are > > Reviewed-by: Matt Turner > > I've requested a change come before patch 1, and then rebased patch 1 > should be an easy R-b. > > I'll need to take a closer look at 11 and

Re: [Mesa-dev] Clamp/saturate optimizations v3

2014-08-19 Thread Matt Turner
On Mon, Aug 18, 2014 at 5:17 AM, Abdiel Janulgue wrote: > v3 of clamp and saturate optimizations Patches 2-4, (5-9 already reviewed), 10, 13-16, (17 already reviewed) are Reviewed-by: Matt Turner I've requested a change come before patch 1, and then rebased patch 1 should be an easy R-b. I'll

[Mesa-dev] Clamp/saturate optimizations v3

2014-08-18 Thread Abdiel Janulgue
v3 of clamp and saturate optimizations Changes since v1: - Only remove the old try_emit_saturate operations after the new optimizations are in place. (Matt, Ian) - Output [min/max](saturate(x),b) instead of saturate([min/max](x,b)) as suggested by Ilia Mirkin. - The change above requir