[PATCH] D64252: [Mips] Instruction `sc` now accepts symbol as an argument

Simon Atanasyan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 7 04:08:49 PDT 2019


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

LGTM with a few nits. I will fix them by myself before committing the patch.



================
Comment at: lib/Target/Mips/MCTargetDesc/MipsTargetStreamer.cpp:348
+    MCOperand &LoOperand, unsigned ATReg, SMLoc IDLoc,
+    const MCSubtargetInfo *STI) {
+  // sc $8, sym => lui $at, %hi(sym)
----------------
clang-format


================
Comment at: lib/Target/Mips/MCTargetDesc/MipsTargetStreamer.cpp:365
+  }
+  else {
+    if (BaseReg != Mips::ZERO)
----------------
ditto


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

https://reviews.llvm.org/D64252





More information about the llvm-commits mailing list