Thanks Eric, that makes sense.<div><br></div><div>I'll respin the patch, confirm no piglit regressions and send it back to the mailing list for your R-b.</div><div><br></div><div>Regards,</div><div>Rhys</div><div><br>On Friday, February 12, 2016, Eric Anholt <<a href="mailto:eric@anholt.net">eric@anholt.net</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Rhys Kidd <<a href="javascript:;" onclick="_e(event, 'cvml', 'rhyskidd@gmail.com')">rhyskidd@gmail.com</a>> writes:<br>
<br>
> Variable no longer is read, and always set to true.<br>
><br>
> Accordingly, the later assert() serves no active current purpose. The prior<br>
> limb of the test 'qinst->cond == QPU_COND_ALWAYS' will never trigger an assert()<br>
> as is. If it did previously define a relevant test, this should be revisited<br>
> in a subsequent patch.<br>
<br>
Looks like just a typo, was supposed to be set to false at the top.<br>
Only the default case handles a cond on the qinst so far, so we want to<br>
be sure that we didn't try to put a cond on any other type of qinst.<br>
</blockquote></div>