[PATCH] D39848: [RISCV] Support lowering FrameIndex

Krzysztof Parzyszek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 16 12:51:44 PST 2017


kparzysz added a comment.

Why do you want the special load/store instructions?



================
Comment at: lib/Target/RISCV/RISCVRegisterInfo.cpp:71
+  unsigned FrameReg = getFrameRegister(MF);
+  int Offset = MF.getFrameInfo().getObjectOffset(FrameIndex) +
+               MI.getOperand(FIOperandNum + 1).getImm();
----------------
You should implement getFrameIndexReference in TargetFrameLowering (in any case).  You could use it here instead of all this logic.


https://reviews.llvm.org/D39848





More information about the llvm-commits mailing list