Re: [Mesa-dev] [PATCH 4/4] glsl: In loop analysis, handle unconditional second assignment.

2013-12-05 Thread Jordan Justen
Series Reviewed-by: Jordan Justen On Thu, Nov 28, 2013 at 11:41 AM, Paul Berry wrote: > Previously, loop analysis would set > this->conditional_or_nested_assignment based on the most recently > visited assignment to the variable. As a result, if a vaiable was > assigned to more than once in a l

[Mesa-dev] [PATCH 4/4] glsl: In loop analysis, handle unconditional second assignment.

2013-11-28 Thread Paul Berry
Previously, loop analysis would set this->conditional_or_nested_assignment based on the most recently visited assignment to the variable. As a result, if a vaiable was assigned to more than once in a loop, the flag might be set incorrectly. For example, in a loop like this: int x; for (i