[PATCH] D10977: Allow {e, r}bp as the target of {read, write}_register.
hfinkel at anl.gov
hfinkel at anl.gov
Tue Jul 7 11:17:49 PDT 2015
================
Comment at: lib/Target/X86/X86ISelLowering.cpp:16231
@@ +16230,3 @@
+ const X86RegisterInfo *RegInfo = Subtarget->getRegisterInfo();
+ unsigned FrameReg =
+ RegInfo->getPtrSizedFrameRegister(DAG.getMachineFunction());
----------------
This variable will be unused in a build without asserts, right? If so, you'll break those builds (many of them also build with -Werror). I think you might as well surround the entire body of this conditional with #ifndef NDEBUG.
Repository:
rL LLVM
http://reviews.llvm.org/D10977
More information about the llvm-commits
mailing list