[PATCH] D139303: [LoongArch] Add custom parser for atomic instructions' memory operand

Lu Weining via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 7 02:00:27 PST 2022


SixWeining accepted this revision.
SixWeining added a comment.
This revision is now accepted and ready to land.

LGTM but let's wait for others.



================
Comment at: llvm/lib/Target/LoongArch/MCTargetDesc/LoongArchInstPrinter.cpp:64
+  const MCOperand &MO = MI->getOperand(OpNo);
+
+  assert(MO.isReg() && "printAtomicMemOp can only print register operands");
----------------
Seems this line is useless.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139303



More information about the llvm-commits mailing list