<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - [r600g] shader failure - r600_sb::bc_finalizer::cf_peephole() when starting Second Life" href="https://bugs.freedesktop.org/show_bug.cgi?id=86089#c2">Comment # 2</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - [r600g] shader failure - r600_sb::bc_finalizer::cf_peephole() when starting Second Life" href="https://bugs.freedesktop.org/show_bug.cgi?id=86089">bug 86089</a> from <a class="email" href="mailto:shawn.starr@rogers.com" title="Shawn Starr <shawn.starr@rogers.com>"> Shawn Starr</a> <pre>Bisecting came up with this as bad commit: 73dd50acf6d244979c2a657906aa56d3ac60d550 is the first bad commit commit 73dd50acf6d244979c2a657906aa56d3ac60d550 Author: Tapani Pälli <<a href="mailto:tapani.palli@intel.com">tapani.palli@intel.com</a>> Date: Wed Aug 6 09:46:54 2014 +0300 glsl: implement switch flow control using a loop Patch removes old variable based logic for handling a break inside switch. Switch is put inside a loop so that existing infrastructure for loop flow control can be used for the switch, now also dead code elimination works properly. Possible 'continue' call inside a switch needs now special handling which is taken care of by detecting continue, breaking out and calling continue for the outside loop. v2: remove one unnecessary ir_expression (Curro) Fixes following Piglit tests: fs-exec-after-break.shader_test fs-conditional-break.shader_test No Piglit or es3conform regressions. Signed-off-by: Tapani Pälli <<a href="mailto:tapani.palli@intel.com">tapani.palli@intel.com</a>> Reviewed-by: Francisco Jerez <<a href="mailto:currojerez@riseup.net">currojerez@riseup.net</a>> :040000 040000 eac8a47066723ff804317ef11bb006dd0bb80d0f b73ed6cc78a66c54d684635f69a358fd54f65929 M src</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>