]> granicus.if.org Git - llvm/commitdiff
[SimplifyCFG] Use hasNUses instead of comparing getNumUses to a constant."
authorCraig Topper <craig.topper@gmail.com>
Mon, 17 Apr 2017 22:13:00 +0000 (22:13 +0000)
committerCraig Topper <craig.topper@gmail.com>
Mon, 17 Apr 2017 22:13:00 +0000 (22:13 +0000)
The use list is a linked list so getNumUses requires a linear scan through the whole list. hasNUses will stop scanning at N and see if that is the end.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@300505 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/Utils/SimplifyCFG.cpp

index 127a44df5344fec7d05023ae201ae481adabb8f9..2f575b9d5027267fef5f380e8e6af262927f1a37 100644 (file)
@@ -3086,7 +3086,7 @@ static bool mergeConditionalStores(BranchInst *PBI, BranchInst *QBI) {
   if ((PTB && !HasOnePredAndOneSucc(PTB, PBI->getParent(), QBI->getParent())) ||
       (QTB && !HasOnePredAndOneSucc(QTB, QBI->getParent(), PostBB)))
     return false;
-  if (PostBB->getNumUses() != 2 || QBI->getParent()->getNumUses() != 2)
+  if (!PostBB->hasNUses(2) || !QBI->getParent()->hasNUses(2))
     return false;
 
   // OK, this is a sequence of two diamonds or triangles.