[PATCH] D39848: [RISCV] Support lowering FrameIndex
Alex Bradbury via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 9 11:49:06 PST 2017
asb added inline comments.
================
Comment at: lib/Target/RISCV/RISCVRegisterInfo.cpp:74
+ unsigned Reg = MI.getOperand(0).getReg();
+ assert(RISCV::GPRRegClass.contains(Reg) && "Unexpected register operand");
+
----------------
mgrang wrote:
> This will have an "unused variable 'Reg'" error in a non-asserts build. No?
No, `Reg` gets used in the code below.
https://reviews.llvm.org/D39848
More information about the llvm-commits
mailing list