<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_RESOLVED bz_closed" title="RESOLVED FIXED - llvmpipe: Fragment Shader with "return" in main causes back output" href="https://bugs.freedesktop.org/show_bug.cgi?id=62357#c15">Comment # 15</a> on <a class="bz_bug_link bz_status_RESOLVED bz_closed" title="RESOLVED FIXED - llvmpipe: Fragment Shader with "return" in main causes back output" href="https://bugs.freedesktop.org/show_bug.cgi?id=62357">bug 62357</a> from <a class="email" href="mailto:jfonseca@vmware.com" title="José Fonseca <jfonseca@vmware.com>"> José Fonseca</a> <pre>(In reply to <a href="show_bug.cgi?id=62357#c14">comment #14</a>) > (In reply to <a href="show_bug.cgi?id=62357#c13">comment #13</a>) > > (In reply to <a href="show_bug.cgi?id=62357#c12">comment #12</a>) > > > This is fixed with 5af7b45986d1b56c568ebe9c3a40d48853e2e9ff. > > > Not sure if the testcase would be that useful as a piglit test. > > > > Why not? > > Yeah hmm maybe it would be useful. > Any ideas where it should be added? No. Choose somwhere you think is good. If people feel strongly about I'm sure they'll voice their opinion on the review request. > It's a crappy test though, Brian's version didn't test both the early exit > taken or not path, whereas my version does but requires glsl 1.30 even > though the bug itself has nothing to do with glsl 1.30. If it tests a bug that was not shown at till now then it is a superb test. Everything else is secondary.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>