diff options
author | Hans Wennborg <hans@hanshq.net> | 2016-05-05 16:39:31 +0000 |
---|---|---|
committer | Hans Wennborg <hans@hanshq.net> | 2016-05-05 16:39:31 +0000 |
commit | 501e739d8a668ddcad2c240f077d0e1f4450a764 (patch) | |
tree | 6d732ea88080aa96aec88452057b838111f44789 /llvm/lib/Target/X86/X86CallFrameOptimization.cpp | |
parent | 462220de478076c326ed66605e2cba11958f5cc3 (diff) | |
download | bcm5719-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.cpp | 15 |
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( |