diff options
author | Bob Wilson <bob.wilson@apple.com> | 2009-12-02 17:15:24 +0000 |
---|---|---|
committer | Bob Wilson <bob.wilson@apple.com> | 2009-12-02 17:15:24 +0000 |
commit | fffbc0c5d92deee5304a4e0b560e15f36f07b944 (patch) | |
tree | 6e350dcd9b5bf6c0198def1ecdcfd76b246ddd49 | |
parent | d831ef494542be07e0faff24306cd032d66ffa26 (diff) | |
download | bcm5719-llvm-fffbc0c5d92deee5304a4e0b560e15f36f07b944.tar.gz bcm5719-llvm-fffbc0c5d92deee5304a4e0b560e15f36f07b944.zip |
Don't count PHI instructions toward the limit for tail duplicating a block.
llvm-svn: 90326
-rw-r--r-- | llvm/lib/CodeGen/TailDuplication.cpp | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/llvm/lib/CodeGen/TailDuplication.cpp b/llvm/lib/CodeGen/TailDuplication.cpp index 9c0b596c33f..a5964585927 100644 --- a/llvm/lib/CodeGen/TailDuplication.cpp +++ b/llvm/lib/CodeGen/TailDuplication.cpp @@ -129,20 +129,22 @@ bool TailDuplicatePass::TailDuplicate(MachineBasicBlock *TailBB, // Check the instructions in the block to determine whether tail-duplication // is invalid or unlikely to be profitable. - unsigned i = 0; + unsigned InstrCount = 0; bool HasCall = false; for (MachineBasicBlock::iterator I = TailBB->begin(); - I != TailBB->end(); ++I, ++i) { + I != TailBB->end(); ++I) { // Non-duplicable things shouldn't be tail-duplicated. if (I->getDesc().isNotDuplicable()) return false; // Don't duplicate more than the threshold. - if (i == MaxDuplicateCount) return false; + if (InstrCount == MaxDuplicateCount) return false; // Remember if we saw a call. if (I->getDesc().isCall()) HasCall = true; + if (I->getOpcode() != TargetInstrInfo::PHI) + InstrCount += 1; } // Heuristically, don't tail-duplicate calls if it would expand code size, // as it's less likely to be worth the extra cost. - if (i > 1 && HasCall) + if (InstrCount > 1 && HasCall) return false; // Iterate through all the unique predecessors and tail-duplicate this |