Re: [Mesa-dev] [PATCH 1/5] glsl: Add conditional-select IR.

2013-09-23 Thread Marek Olšák
No, I've got a patch for Gallium, I will send it in a moment. Marek On Mon, Sep 23, 2013 at 11:25 PM, Matt Turner wrote: > On Mon, Sep 23, 2013 at 2:02 PM, Marek Olšák wrote: >> I think this breaks drivers which use ir_to_mesa, because mix can be >> used in this way in GLSL 1.20 too. >> >> Mare

Re: [Mesa-dev] [PATCH 1/5] glsl: Add conditional-select IR.

2013-09-23 Thread Marek Olšák
Or maybe not. Nevermind. Marek On Mon, Sep 23, 2013 at 11:02 PM, Marek Olšák wrote: > I think this breaks drivers which use ir_to_mesa, because mix can be > used in this way in GLSL 1.20 too. > > Marek > > On Sat, Sep 7, 2013 at 2:57 AM, Matt Turner wrote: >> It's a ?: that operates per-compone

Re: [Mesa-dev] [PATCH 1/5] glsl: Add conditional-select IR.

2013-09-23 Thread Matt Turner
On Mon, Sep 23, 2013 at 2:02 PM, Marek Olšák wrote: > I think this breaks drivers which use ir_to_mesa, because mix can be > used in this way in GLSL 1.20 too. > > Marek Indeed, it's reported as https://bugs.freedesktop.org/show_bug.cgi?id=69202 I was hoping someone would implement it for Galliu

Re: [Mesa-dev] [PATCH 1/5] glsl: Add conditional-select IR.

2013-09-23 Thread Marek Olšák
I think this breaks drivers which use ir_to_mesa, because mix can be used in this way in GLSL 1.20 too. Marek On Sat, Sep 7, 2013 at 2:57 AM, Matt Turner wrote: > It's a ?: that operates per-component on vectors. Will be used in > upcoming lowering pass for ldexp and the implementation of frexp.

Re: [Mesa-dev] [PATCH 1/5] glsl: Add conditional-select IR.

2013-09-08 Thread Ian Romanick
On 09/06/2013 05:57 PM, Matt Turner wrote: > It's a ?: that operates per-component on vectors. Will be used in > upcoming lowering pass for ldexp and the implementation of frexp. > > cond_sel(selector, a, b): >per-component result = selector ? a : b > --- > src/glsl/ir.cpp

[Mesa-dev] [PATCH 1/5] glsl: Add conditional-select IR.

2013-09-06 Thread Matt Turner
It's a ?: that operates per-component on vectors. Will be used in upcoming lowering pass for ldexp and the implementation of frexp. cond_sel(selector, a, b): per-component result = selector ? a : b --- src/glsl/ir.cpp| 2 ++ src/glsl/ir.h