Re: [Mesa-dev] [PATCH] compiler/glsl: Allow the sequence operator to be a constant expression

2016-03-22 Thread Lars Hamre
You are correct, it should be state->is_version(120, 300). I will submit an updated patch. On Tue, Mar 22, 2016 at 3:32 PM, Eduardo Lima Mitev wrote: > On 03/22/2016 02:48 PM, Lars Hamre wrote: > >> Resending this patch because it received no response last week. >> >> Allow

Re: [Mesa-dev] [PATCH] compiler/glsl: Allow the sequence operator to be a constant expression

2016-03-22 Thread Eduardo Lima Mitev
On 03/22/2016 02:48 PM, Lars Hamre wrote: Resending this patch because it received no response last week. Allow the sequence operator to be a constant expression in GLSL ES versions prior to GLSL ES 3.0 Fixes the following piglit test:

[Mesa-dev] [PATCH] compiler/glsl: Allow the sequence operator to be a constant expression

2016-03-22 Thread Lars Hamre
Resending this patch because it received no response last week. Allow the sequence operator to be a constant expression in GLSL ES versions prior to GLSL ES 3.0 Fixes the following piglit test: /all/spec/glsl-es-1.0/compiler/array-sized-by-sequence-in-parenthesis.vert This mirrors the logic

[Mesa-dev] [PATCH] compiler/glsl: Allow the sequence operator to be a constant expression in GLSL ES versions prior to GLSL ES 3.0

2016-03-14 Thread Lars Hamre
Allow the sequence operator to be a constant expression in GLSL ES versions prior to GLSL ES 3.0 Fixes the following piglit test: /all/spec/glsl-es-1.0/compiler/array-sized-by-sequence-in-parenthesis.vert This mirrors the logic from process_initializer() which performs the same check for