Re: [Mesa-dev] [PATCH] glsl: Fix loop bounds detection.

2013-01-07 Thread Eric Anholt
Paul Berry writes: > When analyzing a loop where the loop condition is expressed in the > non-standard order (e.g. "4 > i" instead of "i < 4"), we were > reversing the condition incorrectly, leading to a loop bound that was > off by 1. > > Fixes piglit tests {vs,fs}-loop-bounds-unrolled.shader_te

Re: [Mesa-dev] [PATCH] glsl: Fix loop bounds detection.

2013-01-07 Thread Matt Turner
On Mon, Jan 7, 2013 at 6:23 PM, Paul Berry wrote: > When analyzing a loop where the loop condition is expressed in the > non-standard order (e.g. "4 > i" instead of "i < 4"), we were > reversing the condition incorrectly, leading to a loop bound that was > off by 1. > > Fixes piglit tests {vs,fs}-

Re: [Mesa-dev] [PATCH] glsl: Fix loop bounds detection.

2013-01-07 Thread Kenneth Graunke
On 01/07/2013 06:23 PM, Paul Berry wrote: When analyzing a loop where the loop condition is expressed in the non-standard order (e.g. "4 > i" instead of "i < 4"), we were reversing the condition incorrectly, leading to a loop bound that was off by 1. Fixes piglit tests {vs,fs}-loop-bounds-unroll

[Mesa-dev] [PATCH] glsl: Fix loop bounds detection.

2013-01-07 Thread Paul Berry
When analyzing a loop where the loop condition is expressed in the non-standard order (e.g. "4 > i" instead of "i < 4"), we were reversing the condition incorrectly, leading to a loop bound that was off by 1. Fixes piglit tests {vs,fs}-loop-bounds-unrolled.shader_test. --- src/glsl/loop_controls.