pan/va: Use validate_register_pair for BLEND pack
authorAlyssa Rosenzweig <alyssa@collabora.com>
Thu, 16 Jun 2022 19:16:30 +0000 (15:16 -0400)
committerMarge Bot <emma+marge@anholt.net>
Tue, 21 Jun 2022 22:19:59 +0000 (22:19 +0000)
Instead of open-coding. Noticed by inspection.

Signed-off-by: Alyssa Rosenzweig <alyssa@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17091>

src/panfrost/bifrost/valhall/va_pack.c

index d762e42..159e28f 100644 (file)
@@ -724,17 +724,7 @@ va_pack_instr(const bi_instr *I)
    {
       /* Source 0 - Blend descriptor (64-bit) */
       hex |= ((uint64_t) va_pack_src(I->src[2])) << 0;
-
-      /* Vaidate that it is a 64-bit register pair */
-      assert(I->src[3].type == I->src[2].type);
-
-      if (I->src[2].type == BI_INDEX_REGISTER) {
-         assert(I->src[3].value & 1);
-         assert(I->src[3].value == I->src[2].value + 1);
-      } else {
-         assert(I->src[3].offset & 1);
-         assert(I->src[3].offset == I->src[2].offset + 1);
-      }
+      va_validate_register_pair(I, 2);
 
       /* Target */
       assert((I->branch_offset & 0x7) == 0);