Rhys Kidd <rhysk...@gmail.com> writes:

> Variable no longer is read, and always set to true.
>
> Accordingly, the later assert() serves no active current purpose. The prior
> limb of the test 'qinst->cond == QPU_COND_ALWAYS' will never trigger an 
> assert()
> as is. If it did previously define a relevant test, this should be revisited
> in a subsequent patch.

Looks like just a typo, was supposed to be set to false at the top.
Only the default case handles a cond on the qinst so far, so we want to
be sure that we didn't try to put a cond on any other type of qinst.

Attachment: signature.asc
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to