freedreno/ir3: encode instruction category in opc_t
authorRob Clark <robclark@freedesktop.org>
Sun, 27 Mar 2016 15:40:36 +0000 (11:40 -0400)
committerRob Clark <robclark@freedesktop.org>
Tue, 5 Apr 2016 00:18:18 +0000 (20:18 -0400)
Been on my TODO list for a while.  If nothing else this will make gdb
properly grok the opc_t enum.

This first step preserves ir3_instruction::category (with an added
assert that category matches what is encoded in opc_t).  Next step is
to drop the category field (and arg to ir3_instr_create()), but that
is split into next commit for bisectability and so that we can run
piglit in the intermediate state to flush out any problems.

Signed-off-by: Rob Clark <robclark@freedesktop.org>
src/gallium/drivers/freedreno/ir3/disasm-a3xx.c
src/gallium/drivers/freedreno/ir3/instr-a3xx.h
src/gallium/drivers/freedreno/ir3/ir3.c
src/gallium/drivers/freedreno/ir3/ir3.h
src/gallium/drivers/freedreno/ir3/ir3_compiler_nir.c

index 599872470fc0410c7b1f5bc759f4b32ec226c63e..e29d15682561c608ed461af465e0b678e47320a5 100644 (file)
@@ -243,7 +243,7 @@ static void print_instr_cat2(instr_t *instr)
                        "?6?",
        };
 
-       switch (cat2->opc) {
+       switch (_OPC(2, cat2->opc)) {
        case OPC_CMPS_F:
        case OPC_CMPS_U:
        case OPC_CMPS_S:
@@ -274,7 +274,7 @@ static void print_instr_cat2(instr_t *instr)
                                cat2->src1_abs, false);
        }
 
-       switch (cat2->opc) {
+       switch (_OPC(2, cat2->opc)) {
        case OPC_ABSNEG_F:
        case OPC_ABSNEG_S:
        case OPC_CLZ_B:
@@ -382,34 +382,34 @@ static void print_instr_cat5(instr_t *instr)
        static const struct {
                bool src1, src2, samp, tex;
        } info[0x1f] = {
-                       [OPC_ISAM]     = { true,  false, true,  true,  },
-                       [OPC_ISAML]    = { true,  true,  true,  true,  },
-                       [OPC_ISAMM]    = { true,  false, true,  true,  },
-                       [OPC_SAM]      = { true,  false, true,  true,  },
-                       [OPC_SAMB]     = { true,  true,  true,  true,  },
-                       [OPC_SAML]     = { true,  true,  true,  true,  },
-                       [OPC_SAMGQ]    = { true,  false, true,  true,  },
-                       [OPC_GETLOD]   = { true,  false, true,  true,  },
-                       [OPC_CONV]     = { true,  true,  true,  true,  },
-                       [OPC_CONVM]    = { true,  true,  true,  true,  },
-                       [OPC_GETSIZE]  = { true,  false, false, true,  },
-                       [OPC_GETBUF]   = { false, false, false, true,  },
-                       [OPC_GETPOS]   = { true,  false, false, true,  },
-                       [OPC_GETINFO]  = { false, false, false, true,  },
-                       [OPC_DSX]      = { true,  false, false, false, },
-                       [OPC_DSY]      = { true,  false, false, false, },
-                       [OPC_GATHER4R] = { true,  false, true,  true,  },
-                       [OPC_GATHER4G] = { true,  false, true,  true,  },
-                       [OPC_GATHER4B] = { true,  false, true,  true,  },
-                       [OPC_GATHER4A] = { true,  false, true,  true,  },
-                       [OPC_SAMGP0]   = { true,  false, true,  true,  },
-                       [OPC_SAMGP1]   = { true,  false, true,  true,  },
-                       [OPC_SAMGP2]   = { true,  false, true,  true,  },
-                       [OPC_SAMGP3]   = { true,  false, true,  true,  },
-                       [OPC_DSXPP_1]  = { true,  false, false, false, },
-                       [OPC_DSYPP_1]  = { true,  false, false, false, },
-                       [OPC_RGETPOS]  = { false, false, false, false, },
-                       [OPC_RGETINFO] = { false, false, false, false, },
+                       [opc_op(OPC_ISAM)]     = { true,  false, true,  true,  },
+                       [opc_op(OPC_ISAML)]    = { true,  true,  true,  true,  },
+                       [opc_op(OPC_ISAMM)]    = { true,  false, true,  true,  },
+                       [opc_op(OPC_SAM)]      = { true,  false, true,  true,  },
+                       [opc_op(OPC_SAMB)]     = { true,  true,  true,  true,  },
+                       [opc_op(OPC_SAML)]     = { true,  true,  true,  true,  },
+                       [opc_op(OPC_SAMGQ)]    = { true,  false, true,  true,  },
+                       [opc_op(OPC_GETLOD)]   = { true,  false, true,  true,  },
+                       [opc_op(OPC_CONV)]     = { true,  true,  true,  true,  },
+                       [opc_op(OPC_CONVM)]    = { true,  true,  true,  true,  },
+                       [opc_op(OPC_GETSIZE)]  = { true,  false, false, true,  },
+                       [opc_op(OPC_GETBUF)]   = { false, false, false, true,  },
+                       [opc_op(OPC_GETPOS)]   = { true,  false, false, true,  },
+                       [opc_op(OPC_GETINFO)]  = { false, false, false, true,  },
+                       [opc_op(OPC_DSX)]      = { true,  false, false, false, },
+                       [opc_op(OPC_DSY)]      = { true,  false, false, false, },
+                       [opc_op(OPC_GATHER4R)] = { true,  false, true,  true,  },
+                       [opc_op(OPC_GATHER4G)] = { true,  false, true,  true,  },
+                       [opc_op(OPC_GATHER4B)] = { true,  false, true,  true,  },
+                       [opc_op(OPC_GATHER4A)] = { true,  false, true,  true,  },
+                       [opc_op(OPC_SAMGP0)]   = { true,  false, true,  true,  },
+                       [opc_op(OPC_SAMGP1)]   = { true,  false, true,  true,  },
+                       [opc_op(OPC_SAMGP2)]   = { true,  false, true,  true,  },
+                       [opc_op(OPC_SAMGP3)]   = { true,  false, true,  true,  },
+                       [opc_op(OPC_DSXPP_1)]  = { true,  false, false, false, },
+                       [opc_op(OPC_DSYPP_1)]  = { true,  false, false, false, },
+                       [opc_op(OPC_RGETPOS)]  = { false, false, false, false, },
+                       [opc_op(OPC_RGETINFO)] = { false, false, false, false, },
        };
        instr_cat5_t *cat5 = &instr->cat5;
        int i;
@@ -423,7 +423,7 @@ static void print_instr_cat5(instr_t *instr)
 
        printf(" ");
 
-       switch (cat5->opc) {
+       switch (_OPC(5, cat5->opc)) {
        case OPC_DSXPP_1:
        case OPC_DSYPP_1:
                break;
@@ -488,7 +488,7 @@ static void print_instr_cat6(instr_t *instr)
        memset(&src1, 0, sizeof(src1));
        memset(&src2, 0, sizeof(src2));
 
-       switch (cat6->opc) {
+       switch (_OPC(6, cat6->opc)) {
        case OPC_RESINFO:
        case OPC_RESFMT:
                dst.full  = type_size(cat6->type) == 32;
@@ -519,7 +519,7 @@ static void print_instr_cat6(instr_t *instr)
                break;
        }
 
-       switch (cat6->opc) {
+       switch (_OPC(6, cat6->opc)) {
        case OPC_PREFETCH:
        case OPC_RESINFO:
                break;
@@ -545,7 +545,7 @@ static void print_instr_cat6(instr_t *instr)
        }
        printf(" ");
 
-       switch (cat6->opc) {
+       switch (_OPC(6, cat6->opc)) {
        case OPC_STG:
                sd = 'g';
                break;
@@ -636,7 +636,7 @@ static void print_instr_cat6(instr_t *instr)
        if (ss)
                printf("]");
 
-       switch (cat6->opc) {
+       switch (_OPC(6, cat6->opc)) {
        case OPC_RESINFO:
        case OPC_RESFMT:
                break;
@@ -656,7 +656,7 @@ static const struct opc_info {
        const char *name;
        void (*print)(instr_t *instr);
 } opcs[1 << (3+NOPC_BITS)] = {
-#define OPC(cat, opc, name) [((cat) << NOPC_BITS) | (opc)] = { (cat), (opc), #name, print_instr_cat##cat }
+#define OPC(cat, opc, name) [(opc)] = { (cat), (opc), #name, print_instr_cat##cat }
        /* category 0: */
        OPC(0, OPC_NOP,          nop),
        OPC(0, OPC_BR,           br),
@@ -672,7 +672,7 @@ static const struct opc_info {
        OPC(0, OPC_FLOW_REV,     flow_rev),
 
        /* category 1: */
-       OPC(1, 0, ),
+       OPC(1, OPC_MOV, ),
 
        /* category 2: */
        OPC(2, OPC_ADD_F,        add.f),
@@ -822,8 +822,8 @@ static const struct opc_info {
 #include "ir3.h"
 const char *ir3_instr_name(struct ir3_instruction *instr)
 {
-       if (instr->category == -1) return "??meta??";
-       return opcs[(instr->category << NOPC_BITS) | instr->opc].name;
+       if (opc_cat(instr->opc) == -1) return "??meta??";
+       return opcs[instr->opc].name;
 }
 
 static void print_instr(uint32_t *dwords, int level, int n)
index 1b1f1f0a797f2e4b1203dbd1914f789f92956fd4..87083fd1e81582721cb730a4f75d55ecb6c695ff 100644 (file)
 #include <stdint.h>
 #include <assert.h>
 
+/* size of largest OPC field of all the instruction categories: */
+#define NOPC_BITS 6
+
+#define _OPC(cat, opc)   (((cat) << NOPC_BITS) | opc)
+
 typedef enum {
        /* category 0: */
-       OPC_NOP = 0,
-       OPC_BR = 1,
-       OPC_JUMP = 2,
-       OPC_CALL = 3,
-       OPC_RET = 4,
-       OPC_KILL = 5,
-       OPC_END = 6,
-       OPC_EMIT = 7,
-       OPC_CUT = 8,
-       OPC_CHMASK = 9,
-       OPC_CHSH = 10,
-       OPC_FLOW_REV = 11,
+       OPC_NOP             = _OPC(0, 0),
+       OPC_BR              = _OPC(0, 1),
+       OPC_JUMP            = _OPC(0, 2),
+       OPC_CALL            = _OPC(0, 3),
+       OPC_RET             = _OPC(0, 4),
+       OPC_KILL            = _OPC(0, 5),
+       OPC_END             = _OPC(0, 6),
+       OPC_EMIT            = _OPC(0, 7),
+       OPC_CUT             = _OPC(0, 8),
+       OPC_CHMASK          = _OPC(0, 9),
+       OPC_CHSH            = _OPC(0, 10),
+       OPC_FLOW_REV        = _OPC(0, 11),
 
        /* category 1: */
-       /* no opc.. all category 1 are variants of mov */
+       OPC_MOV             = _OPC(1, 0),
 
        /* category 2: */
-       OPC_ADD_F = 0,
-       OPC_MIN_F = 1,
-       OPC_MAX_F = 2,
-       OPC_MUL_F = 3,
-       OPC_SIGN_F = 4,
-       OPC_CMPS_F = 5,
-       OPC_ABSNEG_F = 6,
-       OPC_CMPV_F = 7,
+       OPC_ADD_F           = _OPC(2, 0),
+       OPC_MIN_F           = _OPC(2, 1),
+       OPC_MAX_F           = _OPC(2, 2),
+       OPC_MUL_F           = _OPC(2, 3),
+       OPC_SIGN_F          = _OPC(2, 4),
+       OPC_CMPS_F          = _OPC(2, 5),
+       OPC_ABSNEG_F        = _OPC(2, 6),
+       OPC_CMPV_F          = _OPC(2, 7),
        /* 8 - invalid */
-       OPC_FLOOR_F = 9,
-       OPC_CEIL_F = 10,
-       OPC_RNDNE_F = 11,
-       OPC_RNDAZ_F = 12,
-       OPC_TRUNC_F = 13,
+       OPC_FLOOR_F         = _OPC(2, 9),
+       OPC_CEIL_F          = _OPC(2, 10),
+       OPC_RNDNE_F         = _OPC(2, 11),
+       OPC_RNDAZ_F         = _OPC(2, 12),
+       OPC_TRUNC_F         = _OPC(2, 13),
        /* 14-15 - invalid */
-       OPC_ADD_U = 16,
-       OPC_ADD_S = 17,
-       OPC_SUB_U = 18,
-       OPC_SUB_S = 19,
-       OPC_CMPS_U = 20,
-       OPC_CMPS_S = 21,
-       OPC_MIN_U = 22,
-       OPC_MIN_S = 23,
-       OPC_MAX_U = 24,
-       OPC_MAX_S = 25,
-       OPC_ABSNEG_S = 26,
+       OPC_ADD_U           = _OPC(2, 16),
+       OPC_ADD_S           = _OPC(2, 17),
+       OPC_SUB_U           = _OPC(2, 18),
+       OPC_SUB_S           = _OPC(2, 19),
+       OPC_CMPS_U          = _OPC(2, 20),
+       OPC_CMPS_S          = _OPC(2, 21),
+       OPC_MIN_U           = _OPC(2, 22),
+       OPC_MIN_S           = _OPC(2, 23),
+       OPC_MAX_U           = _OPC(2, 24),
+       OPC_MAX_S           = _OPC(2, 25),
+       OPC_ABSNEG_S        = _OPC(2, 26),
        /* 27 - invalid */
-       OPC_AND_B = 28,
-       OPC_OR_B = 29,
-       OPC_NOT_B = 30,
-       OPC_XOR_B = 31,
+       OPC_AND_B           = _OPC(2, 28),
+       OPC_OR_B            = _OPC(2, 29),
+       OPC_NOT_B           = _OPC(2, 30),
+       OPC_XOR_B           = _OPC(2, 31),
        /* 32 - invalid */
-       OPC_CMPV_U = 33,
-       OPC_CMPV_S = 34,
+       OPC_CMPV_U          = _OPC(2, 33),
+       OPC_CMPV_S          = _OPC(2, 34),
        /* 35-47 - invalid */
-       OPC_MUL_U = 48,
-       OPC_MUL_S = 49,
-       OPC_MULL_U = 50,
-       OPC_BFREV_B = 51,
-       OPC_CLZ_S = 52,
-       OPC_CLZ_B = 53,
-       OPC_SHL_B = 54,
-       OPC_SHR_B = 55,
-       OPC_ASHR_B = 56,
-       OPC_BARY_F = 57,
-       OPC_MGEN_B = 58,
-       OPC_GETBIT_B = 59,
-       OPC_SETRM = 60,
-       OPC_CBITS_B = 61,
-       OPC_SHB = 62,
-       OPC_MSAD = 63,
+       OPC_MUL_U           = _OPC(2, 48),
+       OPC_MUL_S           = _OPC(2, 49),
+       OPC_MULL_U          = _OPC(2, 50),
+       OPC_BFREV_B         = _OPC(2, 51),
+       OPC_CLZ_S           = _OPC(2, 52),
+       OPC_CLZ_B           = _OPC(2, 53),
+       OPC_SHL_B           = _OPC(2, 54),
+       OPC_SHR_B           = _OPC(2, 55),
+       OPC_ASHR_B          = _OPC(2, 56),
+       OPC_BARY_F          = _OPC(2, 57),
+       OPC_MGEN_B          = _OPC(2, 58),
+       OPC_GETBIT_B        = _OPC(2, 59),
+       OPC_SETRM           = _OPC(2, 60),
+       OPC_CBITS_B         = _OPC(2, 61),
+       OPC_SHB             = _OPC(2, 62),
+       OPC_MSAD            = _OPC(2, 63),
 
        /* category 3: */
-       OPC_MAD_U16 = 0,
-       OPC_MADSH_U16 = 1,
-       OPC_MAD_S16 = 2,
-       OPC_MADSH_M16 = 3,   /* should this be .s16? */
-       OPC_MAD_U24 = 4,
-       OPC_MAD_S24 = 5,
-       OPC_MAD_F16 = 6,
-       OPC_MAD_F32 = 7,
-       OPC_SEL_B16 = 8,
-       OPC_SEL_B32 = 9,
-       OPC_SEL_S16 = 10,
-       OPC_SEL_S32 = 11,
-       OPC_SEL_F16 = 12,
-       OPC_SEL_F32 = 13,
-       OPC_SAD_S16 = 14,
-       OPC_SAD_S32 = 15,
+       OPC_MAD_U16         = _OPC(3, 0),
+       OPC_MADSH_U16       = _OPC(3, 1),
+       OPC_MAD_S16         = _OPC(3, 2),
+       OPC_MADSH_M16       = _OPC(3, 3),   /* should this be .s16? */
+       OPC_MAD_U24         = _OPC(3, 4),
+       OPC_MAD_S24         = _OPC(3, 5),
+       OPC_MAD_F16         = _OPC(3, 6),
+       OPC_MAD_F32         = _OPC(3, 7),
+       OPC_SEL_B16         = _OPC(3, 8),
+       OPC_SEL_B32         = _OPC(3, 9),
+       OPC_SEL_S16         = _OPC(3, 10),
+       OPC_SEL_S32         = _OPC(3, 11),
+       OPC_SEL_F16         = _OPC(3, 12),
+       OPC_SEL_F32         = _OPC(3, 13),
+       OPC_SAD_S16         = _OPC(3, 14),
+       OPC_SAD_S32         = _OPC(3, 15),
 
        /* category 4: */
-       OPC_RCP = 0,
-       OPC_RSQ = 1,
-       OPC_LOG2 = 2,
-       OPC_EXP2 = 3,
-       OPC_SIN = 4,
-       OPC_COS = 5,
-       OPC_SQRT = 6,
+       OPC_RCP             = _OPC(4, 0),
+       OPC_RSQ             = _OPC(4, 1),
+       OPC_LOG2            = _OPC(4, 2),
+       OPC_EXP2            = _OPC(4, 3),
+       OPC_SIN             = _OPC(4, 4),
+       OPC_COS             = _OPC(4, 5),
+       OPC_SQRT            = _OPC(4, 6),
        // 7-63 - invalid
 
        /* category 5: */
-       OPC_ISAM = 0,
-       OPC_ISAML = 1,
-       OPC_ISAMM = 2,
-       OPC_SAM = 3,
-       OPC_SAMB = 4,
-       OPC_SAML = 5,
-       OPC_SAMGQ = 6,
-       OPC_GETLOD = 7,
-       OPC_CONV = 8,
-       OPC_CONVM = 9,
-       OPC_GETSIZE = 10,
-       OPC_GETBUF = 11,
-       OPC_GETPOS = 12,
-       OPC_GETINFO = 13,
-       OPC_DSX = 14,
-       OPC_DSY = 15,
-       OPC_GATHER4R = 16,
-       OPC_GATHER4G = 17,
-       OPC_GATHER4B = 18,
-       OPC_GATHER4A = 19,
-       OPC_SAMGP0 = 20,
-       OPC_SAMGP1 = 21,
-       OPC_SAMGP2 = 22,
-       OPC_SAMGP3 = 23,
-       OPC_DSXPP_1 = 24,
-       OPC_DSYPP_1 = 25,
-       OPC_RGETPOS = 26,
-       OPC_RGETINFO = 27,
+       OPC_ISAM            = _OPC(5, 0),
+       OPC_ISAML           = _OPC(5, 1),
+       OPC_ISAMM           = _OPC(5, 2),
+       OPC_SAM             = _OPC(5, 3),
+       OPC_SAMB            = _OPC(5, 4),
+       OPC_SAML            = _OPC(5, 5),
+       OPC_SAMGQ           = _OPC(5, 6),
+       OPC_GETLOD          = _OPC(5, 7),
+       OPC_CONV            = _OPC(5, 8),
+       OPC_CONVM           = _OPC(5, 9),
+       OPC_GETSIZE         = _OPC(5, 10),
+       OPC_GETBUF          = _OPC(5, 11),
+       OPC_GETPOS          = _OPC(5, 12),
+       OPC_GETINFO         = _OPC(5, 13),
+       OPC_DSX             = _OPC(5, 14),
+       OPC_DSY             = _OPC(5, 15),
+       OPC_GATHER4R        = _OPC(5, 16),
+       OPC_GATHER4G        = _OPC(5, 17),
+       OPC_GATHER4B        = _OPC(5, 18),
+       OPC_GATHER4A        = _OPC(5, 19),
+       OPC_SAMGP0          = _OPC(5, 20),
+       OPC_SAMGP1          = _OPC(5, 21),
+       OPC_SAMGP2          = _OPC(5, 22),
+       OPC_SAMGP3          = _OPC(5, 23),
+       OPC_DSXPP_1         = _OPC(5, 24),
+       OPC_DSYPP_1         = _OPC(5, 25),
+       OPC_RGETPOS         = _OPC(5, 26),
+       OPC_RGETINFO        = _OPC(5, 27),
 
        /* category 6: */
-       OPC_LDG = 0,        /* load-global */
-       OPC_LDL = 1,
-       OPC_LDP = 2,
-       OPC_STG = 3,        /* store-global */
-       OPC_STL = 4,
-       OPC_STP = 5,
-       OPC_STI = 6,
-       OPC_G2L = 7,
-       OPC_L2G = 8,
-       OPC_PREFETCH = 9,
-       OPC_LDLW = 10,
-       OPC_STLW = 11,
-       OPC_RESFMT = 14,
-       OPC_RESINFO = 15,
-       OPC_ATOMIC_ADD = 16,
-       OPC_ATOMIC_SUB = 17,
-       OPC_ATOMIC_XCHG = 18,
-       OPC_ATOMIC_INC = 19,
-       OPC_ATOMIC_DEC = 20,
-       OPC_ATOMIC_CMPXCHG = 21,
-       OPC_ATOMIC_MIN = 22,
-       OPC_ATOMIC_MAX = 23,
-       OPC_ATOMIC_AND = 24,
-       OPC_ATOMIC_OR = 25,
-       OPC_ATOMIC_XOR = 26,
-       OPC_LDGB_TYPED_4D = 27,
-       OPC_STGB_4D_4 = 28,
-       OPC_STIB = 29,
-       OPC_LDC_4 = 30,
-       OPC_LDLV = 31,
+       OPC_LDG             = _OPC(6, 0),        /* load-global */
+       OPC_LDL             = _OPC(6, 1),
+       OPC_LDP             = _OPC(6, 2),
+       OPC_STG             = _OPC(6, 3),        /* store-global */
+       OPC_STL             = _OPC(6, 4),
+       OPC_STP             = _OPC(6, 5),
+       OPC_STI             = _OPC(6, 6),
+       OPC_G2L             = _OPC(6, 7),
+       OPC_L2G             = _OPC(6, 8),
+       OPC_PREFETCH        = _OPC(6, 9),
+       OPC_LDLW            = _OPC(6, 10),
+       OPC_STLW            = _OPC(6, 11),
+       OPC_RESFMT          = _OPC(6, 14),
+       OPC_RESINFO         = _OPC(6, 15),
+       OPC_ATOMIC_ADD      = _OPC(6, 16),
+       OPC_ATOMIC_SUB      = _OPC(6, 17),
+       OPC_ATOMIC_XCHG     = _OPC(6, 18),
+       OPC_ATOMIC_INC      = _OPC(6, 19),
+       OPC_ATOMIC_DEC      = _OPC(6, 20),
+       OPC_ATOMIC_CMPXCHG  = _OPC(6, 21),
+       OPC_ATOMIC_MIN      = _OPC(6, 22),
+       OPC_ATOMIC_MAX      = _OPC(6, 23),
+       OPC_ATOMIC_AND      = _OPC(6, 24),
+       OPC_ATOMIC_OR       = _OPC(6, 25),
+       OPC_ATOMIC_XOR      = _OPC(6, 26),
+       OPC_LDGB_TYPED_4D   = _OPC(6, 27),
+       OPC_STGB_4D_4       = _OPC(6, 28),
+       OPC_STIB            = _OPC(6, 29),
+       OPC_LDC_4           = _OPC(6, 30),
+       OPC_LDLV            = _OPC(6, 31),
 
        /* meta instructions (category -1): */
        /* placeholder instr to mark shader inputs: */
-       OPC_META_INPUT = 0,
-       OPC_META_PHI = 1,
+       OPC_META_INPUT      = _OPC(-1, 0),
+       OPC_META_PHI        = _OPC(-1, 1),
        /* The "fan-in" and "fan-out" instructions are used for keeping
         * track of instructions that write to multiple dst registers
         * (fan-out) like texture sample instructions, or read multiple
         * consecutive scalar registers (fan-in) (bary.f, texture samp)
         */
-       OPC_META_FO = 2,
-       OPC_META_FI = 3,
+       OPC_META_FO         = _OPC(-1, 2),
+       OPC_META_FI         = _OPC(-1, 3),
 
 } opc_t;
 
+#define opc_cat(opc) ((int)((opc) >> NOPC_BITS))
+#define opc_op(opc)  ((unsigned)((opc) & ((1 << NOPC_BITS) - 1)))
+
 typedef enum {
        TYPE_F16 = 0,
        TYPE_F32 = 1,
@@ -472,7 +480,7 @@ typedef struct PACKED {
 
 static inline bool instr_cat3_full(instr_cat3_t *cat3)
 {
-       switch (cat3->opc) {
+       switch (_OPC(3, cat3->opc)) {
        case OPC_MAD_F16:
        case OPC_MAD_U16:
        case OPC_MAD_S16:
index 7d89142d7a13148ed805fbc32d0b78f5a26a202e..d86dfca9d54d11477d0411ac9f8dc015993efa36 100644 (file)
@@ -688,6 +688,7 @@ struct ir3_instruction * ir3_instr_create2(struct ir3_block *block,
        struct ir3_instruction *instr = instr_create(block, nreg);
        instr->block = block;
        instr->category = category;
+       debug_assert(opc_cat(opc) == category);
        instr->opc = opc;
        insert_instr(block, instr);
        return instr;
index 1a109d880e6546b8f7294e7535295ab42771ede8..c9a9d29faf79c240958c63488abdff5f5ce61f8e 100644 (file)
@@ -902,7 +902,7 @@ static inline struct ir3_instruction *
 ir3_MOV(struct ir3_block *block, struct ir3_instruction *src, type_t type)
 {
        struct ir3_instruction *instr =
-               ir3_instr_create(block, 1, 0);
+               ir3_instr_create(block, 1, OPC_MOV);
        ir3_reg_create(instr, 0, 0);   /* dst */
        if (src->regs[0]->flags & IR3_REG_ARRAY) {
                struct ir3_register *src_reg =
@@ -923,7 +923,7 @@ ir3_COV(struct ir3_block *block, struct ir3_instruction *src,
                type_t src_type, type_t dst_type)
 {
        struct ir3_instruction *instr =
-               ir3_instr_create(block, 1, 0);
+               ir3_instr_create(block, 1, OPC_MOV);
        ir3_reg_create(instr, 0, 0);   /* dst */
        ir3_reg_create(instr, 0, IR3_REG_SSA)->instr = src;
        instr->cat1.src_type = src_type;
index 3d656d4a34df0f73c211caf7c041080d08fc234c..fafd6beea4bfce2ad975dd45884164330823aa16 100644 (file)
@@ -286,7 +286,7 @@ create_immed(struct ir3_block *block, uint32_t val)
 {
        struct ir3_instruction *mov;
 
-       mov = ir3_instr_create(block, 1, 0);
+       mov = ir3_instr_create(block, 1, OPC_MOV);
        mov->cat1.src_type = TYPE_U32;
        mov->cat1.dst_type = TYPE_U32;
        ir3_reg_create(mov, 0, 0);
@@ -366,7 +366,7 @@ create_uniform(struct ir3_compile *ctx, unsigned n)
 {
        struct ir3_instruction *mov;
 
-       mov = ir3_instr_create(ctx->block, 1, 0);
+       mov = ir3_instr_create(ctx->block, 1, OPC_MOV);
        /* TODO get types right? */
        mov->cat1.src_type = TYPE_F32;
        mov->cat1.dst_type = TYPE_F32;
@@ -382,7 +382,7 @@ create_uniform_indirect(struct ir3_compile *ctx, int n,
 {
        struct ir3_instruction *mov;
 
-       mov = ir3_instr_create(ctx->block, 1, 0);
+       mov = ir3_instr_create(ctx->block, 1, OPC_MOV);
        mov->cat1.src_type = TYPE_U32;
        mov->cat1.dst_type = TYPE_U32;
        ir3_reg_create(mov, 0, 0);
@@ -418,7 +418,7 @@ create_indirect_load(struct ir3_compile *ctx, unsigned arrsz, int n,
        struct ir3_instruction *mov;
        struct ir3_register *src;
 
-       mov = ir3_instr_create(block, 1, 0);
+       mov = ir3_instr_create(block, 1, OPC_MOV);
        mov->cat1.src_type = TYPE_U32;
        mov->cat1.dst_type = TYPE_U32;
        ir3_reg_create(mov, 0, 0);
@@ -441,7 +441,7 @@ create_var_load(struct ir3_compile *ctx, struct ir3_array *arr, int n,
        struct ir3_instruction *mov;
        struct ir3_register *src;
 
-       mov = ir3_instr_create(block, 1, 0);
+       mov = ir3_instr_create(block, 1, OPC_MOV);
        mov->cat1.src_type = TYPE_U32;
        mov->cat1.dst_type = TYPE_U32;
        ir3_reg_create(mov, 0, 0);
@@ -469,7 +469,7 @@ create_var_store(struct ir3_compile *ctx, struct ir3_array *arr, int n,
        struct ir3_instruction *mov;
        struct ir3_register *dst;
 
-       mov = ir3_instr_create(block, 1, 0);
+       mov = ir3_instr_create(block, 1, OPC_MOV);
        mov->cat1.src_type = TYPE_U32;
        mov->cat1.dst_type = TYPE_U32;
        dst = ir3_reg_create(mov, 0, IR3_REG_ARRAY |