pan/midgard: Reorder bits check to fix 8-bit masks

Coverity.

Signed-off-by: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
This commit is contained in:
Alyssa Rosenzweig 2019-08-21 09:17:03 -07:00
parent 6189274f57
commit 91a5b2657d

View file

@ -411,6 +411,11 @@ print_mask_vec16(uint8_t mask, midgard_dest_override override)
static void
print_mask(uint8_t mask, unsigned bits, midgard_dest_override override)
{
if (bits == 8) {
print_mask_vec16(mask, override);
return;
}
if (bits < 16) {
/* Shouldn't happen but with junk / out-of-spec shaders it
* would cause an infinite loop */
@ -419,11 +424,6 @@ print_mask(uint8_t mask, unsigned bits, midgard_dest_override override)
return;
}
if (bits == 8) {
print_mask_vec16(mask, override);
return;
}
/* Skip 'complete' masks */
if (bits >= 32 && mask == 0xFF) return;