Re: [Mesa-dev] [PATCH] glsl/dead_builin_varyings: Fix gl_FragData array lowering

2015-11-30 Thread Tapani Pälli
On 11/30/2015 10:14 AM, Iago Toral wrote: On Mon, 2015-11-30 at 09:56 +0200, Tapani Pälli wrote: Hi; On 11/27/2015 02:14 PM, Iago Toral Quiroga wrote: The current implementation looks for array dereferences on gl_FragData and immediately then proceeds to lower them, however this is not enoug

Re: [Mesa-dev] [PATCH] glsl/dead_builin_varyings: Fix gl_FragData array lowering

2015-11-30 Thread Iago Toral
On Mon, 2015-11-30 at 09:56 +0200, Tapani Pälli wrote: > Hi; > > On 11/27/2015 02:14 PM, Iago Toral Quiroga wrote: > > The current implementation looks for array dereferences on gl_FragData and > > immediately then proceeds to lower them, however this is not enough because > > we can have array ac

Re: [Mesa-dev] [PATCH] glsl/dead_builin_varyings: Fix gl_FragData array lowering

2015-11-29 Thread Tapani Pälli
Hi; On 11/27/2015 02:14 PM, Iago Toral Quiroga wrote: The current implementation looks for array dereferences on gl_FragData and immediately then proceeds to lower them, however this is not enough because we can have array access on vector variables too, like in this code: out vec4 color; void

[Mesa-dev] [PATCH] glsl/dead_builin_varyings: Fix gl_FragData array lowering

2015-11-27 Thread Iago Toral Quiroga
The current implementation looks for array dereferences on gl_FragData and immediately then proceeds to lower them, however this is not enough because we can have array access on vector variables too, like in this code: out vec4 color; void main() { int i; for (i = 0; i < 4; i++) color