[PATCH] D125824: [AMDGPU] gfx11 export instructions

[Jay Foad via Phabricator via llvm-commits]

foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/Disassembler/AMDGPUDisassembler.cpp:642-643
+  if (STI.getFeatureBits()[AMDGPU::FeatureGFX11]) {
+    // The MCInst still has these fields even though they are no longer encoded
+    // in the GFX11 instruction.
+    insertNamedMCOperand(MI, MCOperand::createImm(0), AMDGPU::OpName::vm);
----------------
arsenm wrote:
> Why not remove them then? I guess that would just move the handling from MI->MC from the pseudo
Probably just because I didn't know how to implement that. Is there a good place to do stuff like this when lowering from MI to MC?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125824