summaryrefslogtreecommitdiffstats
path: root/llvm/lib/Target/X86/X86CallFrameOptimization.cpp
diff options
context:
space:
mode:
authorHans Wennborg <hans@hanshq.net>2016-05-05 16:39:31 +0000
committerHans Wennborg <hans@hanshq.net>2016-05-05 16:39:31 +0000
commit501e739d8a668ddcad2c240f077d0e1f4450a764 (patch)
tree6d732ea88080aa96aec88452057b838111f44789 /llvm/lib/Target/X86/X86CallFrameOptimization.cpp
parent462220de478076c326ed66605e2cba11958f5cc3 (diff)
downloadbcm5719-llvm-501e739d8a668ddcad2c240f077d0e1f4450a764.tar.gz
bcm5719-llvm-501e739d8a668ddcad2c240f077d0e1f4450a764.zip
X86CallFrameOptimization: make adjustCallSequence's return type void
It always returned the same value (true). No functionality change. llvm-svn: 268645
Diffstat (limited to 'llvm/lib/Target/X86/X86CallFrameOptimization.cpp')
-rw-r--r--llvm/lib/Target/X86/X86CallFrameOptimization.cpp15
1 files changed, 8 insertions, 7 deletions
diff --git a/llvm/lib/Target/X86/X86CallFrameOptimization.cpp b/llvm/lib/Target/X86/X86CallFrameOptimization.cpp
index ac71fc2e5bd..367ae4f4b5a 100644
--- a/llvm/lib/Target/X86/X86CallFrameOptimization.cpp
+++ b/llvm/lib/Target/X86/X86CallFrameOptimization.cpp
@@ -88,7 +88,7 @@ private:
void collectCallInfo(MachineFunction &MF, MachineBasicBlock &MBB,
MachineBasicBlock::iterator I, CallContext &Context);
- bool adjustCallSequence(MachineFunction &MF, const CallContext &Context);
+ void adjustCallSequence(MachineFunction &MF, const CallContext &Context);
MachineInstr *canFoldIntoRegPush(MachineBasicBlock::iterator FrameSetup,
unsigned Reg);
@@ -245,9 +245,12 @@ bool X86CallFrameOptimization::runOnMachineFunction(MachineFunction &MF) {
if (!isProfitable(MF, CallSeqVector))
return false;
- for (auto CC : CallSeqVector)
- if (CC.UsePush)
- Changed |= adjustCallSequence(MF, CC);
+ for (auto CC : CallSeqVector) {
+ if (CC.UsePush) {
+ adjustCallSequence(MF, CC);
+ Changed = true;
+ }
+ }
return Changed;
}
@@ -445,7 +448,7 @@ void X86CallFrameOptimization::collectCallInfo(MachineFunction &MF,
Context.UsePush = true;
}
-bool X86CallFrameOptimization::adjustCallSequence(MachineFunction &MF,
+void X86CallFrameOptimization::adjustCallSequence(MachineFunction &MF,
const CallContext &Context) {
// Ok, we can in fact do the transformation for this call.
// Do not remove the FrameSetup instruction, but adjust the parameters.
@@ -543,8 +546,6 @@ bool X86CallFrameOptimization::adjustCallSequence(MachineFunction &MF,
// frame.
X86MachineFunctionInfo *FuncInfo = MF.getInfo<X86MachineFunctionInfo>();
FuncInfo->setHasPushSequences(true);
-
- return true;
}
MachineInstr *X86CallFrameOptimization::canFoldIntoRegPush(
OpenPOWER on IntegriCloud