maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

This patch fixes the invalid usage of OptSize in Machine Combiner. 

Differential Revision: https://reviews.llvm.org/D43813

llvm-svn: 327721
This commit is contained in:
Andrew V. Tischenko 2018-03-16 16:06:24 +00:00
parent 9c1a6e7e24
commit a0cd09d4a2
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
llvm/lib/CodeGen/MachineCombiner.cpp
View File

@ -407,8 +407,8 @@ bool MachineCombiner::preservesResourceLen(
/// \returns true when new instruction sequence should be generated
/// independent if it lengthens critical path or not
bool MachineCombiner::doSubstitute(unsigned NewSize, unsigned OldSize) {
if (OptSize && (NewSize < OldSize))
return true;
if (OptSize) // First of all check OptSize option
return (NewSize < OldSize); // Only substitute if new size < old size
if (!TSchedModel.hasInstrSchedModelOrItineraries())
return true;
return false;
@ -588,7 +588,7 @@ bool MachineCombiner::combineInstructions(MachineBasicBlock *MBB) {
// Eagerly stop after the first pattern fires.
Changed = true;
break;
} else {
} else if (!OptSize || (NewInstCount <= OldInstCount)) {
// For big basic blocks, we only compute the full trace the first time
// we hit this. We do not invalidate the trace, but instead update the
// instruction depths incrementally.