[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