[PATCH] D128187: [RISCV] Add ADDI instr to compute FrameIndex address for RVV

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 26 11:15:51 PDT 2022


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelDAGToDAG.cpp:1823
 
 bool RISCVDAGToDAGISel::SelectBaseAddr(SDValue Addr, SDValue &Base) {
   // If this is FrameIndex, select it directly. Otherwise just let it get
----------------
I did some refactoring in SelectAddrRegImm to remove the dependency on SelectBaseAddr. You can remove it in this patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128187



More information about the llvm-commits mailing list