[PATCH] D122263: [AMDGPU] gfx940 MAI hazard recognizer

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/GCNHazardRecognizer.cpp:115
+
+  switch (Opcode) {
+  case AMDGPU::V_MFMA_F32_16X16X8XF32_e64:
----------------
Joe_Nash wrote:
> Is this equivalent to isMAI? If not, perhaps you could use a GenericTable to create a lookup if an instruction is relevant here, and mark the instructions in tablegen. It would be more maintainable.
It is not the same. It selects a subset from MAI. The problem with marking it in the td is that the same instructions are executed on different ALUs depending on the subtarget, so it is not just a single bit.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122263/new/

https://reviews.llvm.org/D122263