Re: [Mesa-dev] [PATCH v2 6/7] i915: Don't emit saturate for vertex shaders

2014-12-03 Thread Ian Romanick
On 12/01/2014 11:27 AM, Matt Turner wrote: > On Mon, Dec 1, 2014 at 10:50 AM, Matt Turner wrote: >> I think i915's vertex shaders can use saturate. I don't think this is needed. >> >> The code you were modifying (in Gallium) in the original patch doesn't >> affect this driver. > > Actually, i915

Re: [Mesa-dev] [PATCH v2 6/7] i915: Don't emit saturate for vertex shaders

2014-12-01 Thread Matt Turner
On Mon, Dec 1, 2014 at 10:50 AM, Matt Turner wrote: > I think i915's vertex shaders can use saturate. I don't think this is needed. > > The code you were modifying (in Gallium) in the original patch doesn't > affect this driver. Actually, i915 doesn't do hardware vertex shaders, so I don't know w

Re: [Mesa-dev] [PATCH v2 6/7] i915: Don't emit saturate for vertex shaders

2014-12-01 Thread Matt Turner
I think i915's vertex shaders can use saturate. I don't think this is needed. The code you were modifying (in Gallium) in the original patch doesn't affect this driver. ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org

[Mesa-dev] [PATCH v2 6/7] i915: Don't emit saturate for vertex shaders

2014-12-01 Thread Abdiel Janulgue
Signed-off-by: Abdiel Janulgue --- src/mesa/drivers/dri/i915/i915_context.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/mesa/drivers/dri/i915/i915_context.c b/src/mesa/drivers/dri/i915/i915_context.c index 42ea54e..847105a 100644 --- a/src/mesa/drivers/dri/i915/i915_context.c +++ b