<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 15 February 2016 at 20:17, Eric Anholt <span dir="ltr"><<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Rhys Kidd <<a href="mailto:rhyskidd@gmail.com">rhyskidd@gmail.com</a>> writes:<br>
<br>
> Variable was previously always set to true. Accordingly, the later<br>
> assert() served no active purpose.<br>
><br>
> Found with GCC warning and code inspection:<br>
><br>
> mesa/src/gallium/drivers/vc4/vc4_qpu_emit.c: In function'vc4_generate_code':<br>
> mesa/src/gallium/drivers/vc4/vc4_qpu_emit.c:315:22: warning: variable 'handled_qinst_cond' set but not used [-Wunused-but-set-variable]<br>
>                  bool handled_qinst_cond = true;<br>
>                       ^<br>
><br>
> Separately, the early break for MOV no-ops in the default switch case<br>
> will trigger the assert() in debug builds.<br>
><br>
>     ...<br>
>     /* Skip emitting the MOV if it's a no-op. */<br>
>     if (qir_is_raw_mov(qinst) &&<br>
>         dst.mux == src[0].mux && dst.addr == src[0].addr) {<br>
>             break;<br>
>     }<br>
>     ...<br>
><br>
> This code path is tickled now that this typo is corrected.<br>
<br>
</span>Thanks!  I've pushed your 3 patches, but with this last bit about the<br>
is_raw_mov() problem removed since I just put in a patch to fix the<br>
problem instead :)<br>
</blockquote></div><br></div><div class="gmail_extra">Thanks Eric!<br></div></div>