[PATCH] D123265: [RISCV] Extend the Merge Base Offset pass to handle AUIPC+ADDI

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.
Herald added a subscriber: shiva0217.


================
Comment at: llvm/lib/Target/RISCV/RISCVMergeBaseOffset.cpp:275
+    } else {
+      Tail.removeOperand(2);
+      Tail.addOperand(Lo.getOperand(2));
----------------
Do we need to check that the immediate operand of the load/store is 0 before we remove it?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123265