[PATCH] Fully fix Bug #22115.

Justin Hibbits jrh29 at alumni.cwru.edu
Fri Jan 9 15:26:55 PST 2015


================
Comment at: lib/Target/PowerPC/PPCFrameLowering.cpp:1163
@@ +1162,3 @@
+  // Only used in SVR4 32-bit.
+  int PBPSI = FI->getPICBasePointerSaveIndex();
+  if (FI->usesPICBase()) {
----------------
jhibbits wrote:
> hfinkel wrote:
> > This variable is only used inside the if statement. Why are you creating it out here?
> It's how the others do it (BPSI is identical, which is where I copied it from).  I'll fix this one though.
D'oh, bad reading on my part.  Ignore this.  BPSI is used in the condition.

http://reviews.llvm.org/D6906

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list