[llvm] 82420b4 - [DivRemPairs] Use DenseMapBase::find instead of operator[]. NFC

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 27 01:13:23 PDT 2020


Author: Fangrui Song
Date: 2020-09-27T01:13:14-07:00
New Revision: 82420b4e49ff92c49c2b548bf541a5655e97d197

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

LOG: [DivRemPairs] Use DenseMapBase::find instead of operator[]. NFC

Added: 
    

Modified: 
    llvm/lib/Transforms/Scalar/DivRemPairs.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/Scalar/DivRemPairs.cpp b/llvm/lib/Transforms/Scalar/DivRemPairs.cpp
index d44a5979a8b2..42a5fb1135a6 100644
--- a/llvm/lib/Transforms/Scalar/DivRemPairs.cpp
+++ b/llvm/lib/Transforms/Scalar/DivRemPairs.cpp
@@ -151,8 +151,8 @@ static DivRemWorklistTy getWorklist(Function &F) {
   // rare than division.
   for (auto &RemPair : RemMap) {
     // Find the matching division instruction from the division map.
-    Instruction *DivInst = DivMap[RemPair.first];
-    if (!DivInst)
+    auto It = DivMap.find(RemPair.first);
+    if (It == DivMap.end())
       continue;
 
     // We have a matching pair of div/rem instructions.
@@ -160,7 +160,7 @@ static DivRemWorklistTy getWorklist(Function &F) {
     Instruction *RemInst = RemPair.second;
 
     // Place it in the worklist.
-    Worklist.emplace_back(DivInst, RemInst);
+    Worklist.emplace_back(It->second, RemInst);
   }
 
   return Worklist;


        


More information about the llvm-commits mailing list