intel/gen12: Take into account opcode when decoding SWSB
authorCaio Marcelo de Oliveira Filho <caio.oliveira@intel.com>
Fri, 31 Jan 2020 18:20:25 +0000 (10:20 -0800)
committerCaio Marcelo de Oliveira Filho <caio.oliveira@intel.com>
Tue, 18 Feb 2020 17:17:51 +0000 (09:17 -0800)
commit79788b8f7f07460af8467931501380e47b485e36
tree7cc1d7d73f57e08718d1f4fdf80a70a107a33e36
parentbee5c9b0dc13dbae0ccf124124eaccebf7f2a435
intel/gen12: Take into account opcode when decoding SWSB

The interpretation of the fields is different depending whether the
instruction is a SEND/MATH or not.

This fixes the disassembly output for non-SEND/MATH instructions that
have both in-order and out-of-order dependencies.  Their dependencies
were wrongly represented as `@A $B` when the correct would be `@A
$B.dst`.

Fixes: 6154cdf924f ("intel/eu/gen12: Add auxiliary type to represent SWSB information during codegen.")
Fixes: 83612c01271 ("intel/disasm/gen12: Disassemble software scoreboard information.")
Acked-by: Francisco Jerez <currojerez@riseup.net>
Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3660>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3660>
src/intel/compiler/brw_disasm.c
src/intel/compiler/brw_eu_defines.h