[PATCH] D130345: [AMDGPU][MC][GFX11] Correct encoding of VOP3/VOP3_DPP v_cmpx* opcodes

Carl Ritson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 23 23:44:51 PDT 2022


critson added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/MCTargetDesc/SIMCCodeEmitter.cpp:342
+    assert((Encoding & 0xFF) == 0);
+    Encoding |= 0x7E;
+  }
----------------
Could use the following instead to avoid numeric in code?

Encoding |= MRI.getEncodingValue(AMDGPU::EXEC_LO)


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

https://reviews.llvm.org/D130345



More information about the llvm-commits mailing list