[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