diff options
author | Craig Topper <craig.topper@intel.com> | 2018-10-01 23:09:52 +0000 |
---|---|---|
committer | Craig Topper <craig.topper@intel.com> | 2018-10-01 23:09:52 +0000 |
commit | 22336491c25ccc2e17db4ee02a660bce35f8d01b (patch) | |
tree | 60730bb07a043c4487abbc49057ed6ca62b40737 | |
parent | 740cece063097c856a6edeb0ff55d5178d4baca1 (diff) |
[SimplifyCFG] Use Value::hasNUses instead of 'getNumUses() =='. NFCI
getNumUses is linear in the number of uses. Since we're looking for a specific use count, we can use hasNUses which will stop as soon as it determines there are more than N uses instead of walking all of them.
-rw-r--r-- | llvm/lib/Transforms/Utils/SimplifyCFG.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp index c5bd173df18..458413d2f8a 100644 --- a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp +++ b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp @@ -2001,7 +2001,7 @@ static bool SpeculativelyExecuteBB(BranchInst *BI, BasicBlock *ThenBB, I = SinkCandidateUseCounts.begin(), E = SinkCandidateUseCounts.end(); I != E; ++I) - if (I->first->getNumUses() == I->second) { + if (I->first->hasNUses(I->second)) { ++SpeculationCost; if (SpeculationCost > 1) return false; |