intel/disasm: Don't disassemble saturate control on SEND instructions.
authorFrancisco Jerez <currojerez@riseup.net>
Wed, 28 Aug 2019 05:02:19 +0000 (22:02 -0700)
committerFrancisco Jerez <currojerez@riseup.net>
Fri, 11 Oct 2019 19:24:16 +0000 (12:24 -0700)
The field is gone on Gen12+ and it was illegal on previous
generations.

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/intel/compiler/brw_disasm.c

index 594ea665a9c2a7c009a5e73bd0e5f66a7935683c..44c6995d7fd8bd0cf91c6405f999ba424227d2ac 100644 (file)
@@ -1663,8 +1663,10 @@ brw_disassemble_inst(FILE *file, const struct gen_device_info *devinfo,
    }
 
    err |= print_opcode(file, devinfo, opcode);
-   err |= control(file, "saturate", saturate, brw_inst_saturate(devinfo, inst),
-                  NULL);
+
+   if (!is_send(opcode))
+      err |= control(file, "saturate", saturate, brw_inst_saturate(devinfo, inst),
+                     NULL);
 
    err |= control(file, "debug control", debug_ctrl,
                   brw_inst_debug_control(devinfo, inst), NULL);