[llvm] [AMDGPU] Remove AtomicNoRet class and getAtomicNoRetOp table (PR #83593)

Jay Foad via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 1 08:51:03 PST 2024


================
@@ -8168,20 +8168,6 @@ void AMDGPUAsmParser::cvtMubufImpl(MCInst &Inst,
   bool IsAtomicReturn = false;
 
   if (IsAtomic) {
-    for (unsigned i = FirstOperandIdx, e = Operands.size(); i != e; ++i) {
-      AMDGPUOperand &Op = ((AMDGPUOperand &)*Operands[i]);
-      if (!Op.isCPol())
-        continue;
-      IsAtomicReturn = Op.getImm() & AMDGPU::CPol::GLC;
-      break;
-    }
-
-    if (!IsAtomicReturn) {
-      int NewOpc = AMDGPU::getAtomicNoRetOp(Inst.getOpcode());
-      if (NewOpc != -1)
-        Inst.setOpcode(NewOpc);
-    }
-
----------------
jayfoad wrote:

I don't understand what this code was for. Everything still seems to work without it.

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


More information about the llvm-commits mailing list