[llvm] [AMDGPU][True16][MC] fix opsel for v_cmpx 16bit inst (PR #135441)

Brox Chen via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 14 11:42:40 PDT 2025


================
@@ -1220,6 +1220,10 @@ void AMDGPUInstPrinter::printPackedModifier(const MCInst *MI,
         (ModIdx != -1) ? MI->getOperand(ModIdx).getImm() : DefaultValue;
   }
 
+  const bool HasDst =
+      (AMDGPU::getNamedOperandIdx(Opc, AMDGPU::OpName::vdst) != -1) ||
+      (AMDGPU::getNamedOperandIdx(Opc, AMDGPU::OpName::sdst) != -1);
----------------
broxigarchen wrote:

Hi Matt are you referring to the isDef check for the machineoperand? This is used in asm/dasm for printing and I thought we don't have the context there to check if an operand is a def?

https://github.com/llvm/llvm-project/pull/135441


More information about the llvm-commits mailing list