[llvm] caebb45 - [Transforms] Avoid repeated hash looksup (NFC) (#107727)

via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 7 18:16:09 PDT 2024


Author: Kazu Hirata
Date: 2024-09-07T18:16:06-07:00
New Revision: caebb4562ce634a22f7b13480b19cffc2a6a6730

URL: https://github.com/llvm/llvm-project/commit/caebb4562ce634a22f7b13480b19cffc2a6a6730
DIFF: https://github.com/llvm/llvm-project/commit/caebb4562ce634a22f7b13480b19cffc2a6a6730.diff

LOG: [Transforms] Avoid repeated hash looksup (NFC) (#107727)

Added: 
    

Modified: 
    llvm/lib/Transforms/Utils/LoopUnrollAndJam.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/Utils/LoopUnrollAndJam.cpp b/llvm/lib/Transforms/Utils/LoopUnrollAndJam.cpp
index c7b88d3c48a69f..2c2400d9dd7a82 100644
--- a/llvm/lib/Transforms/Utils/LoopUnrollAndJam.cpp
+++ b/llvm/lib/Transforms/Utils/LoopUnrollAndJam.cpp
@@ -139,11 +139,9 @@ static bool processHeaderPhiOperands(BasicBlock *Header, BasicBlock *Latch,
   SmallPtrSet<Instruction *, 8> VisitedInstr;
 
   std::function<bool(Instruction * I)> ProcessInstr = [&](Instruction *I) {
-    if (VisitedInstr.count(I))
+    if (!VisitedInstr.insert(I).second)
       return true;
 
-    VisitedInstr.insert(I);
-
     if (AftBlocks.count(I->getParent()))
       for (auto &U : I->operands())
         if (Instruction *II = dyn_cast<Instruction>(U))


        


More information about the llvm-commits mailing list