[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