[llvm] 8b9c412 - [DominanceFrontier] fix addToFrontier to use insert

Bing1 Yu via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 2 22:58:50 PDT 2023


Author: Wenju He
Date: 2023-08-03T13:58:37+08:00
New Revision: 8b9c412aec413fb073b960569b844e989ee32491

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

LOG: [DominanceFrontier] fix addToFrontier to use insert

No new lits are added since addToFrontier is only used in MachineDominanceFrontier::addToFrontier.

Reviewed By: arsenm

Differential Revision: https://reviews.llvm.org/D155417

Added: 
    

Modified: 
    llvm/include/llvm/Analysis/DominanceFrontierImpl.h

Removed: 
    


################################################################################
diff  --git a/llvm/include/llvm/Analysis/DominanceFrontierImpl.h b/llvm/include/llvm/Analysis/DominanceFrontierImpl.h
index 7a5f8f31bae392..3df51d9ad90815 100644
--- a/llvm/include/llvm/Analysis/DominanceFrontierImpl.h
+++ b/llvm/include/llvm/Analysis/DominanceFrontierImpl.h
@@ -57,8 +57,7 @@ template <class BlockT, bool IsPostDom>
 void DominanceFrontierBase<BlockT, IsPostDom>::addToFrontier(iterator I,
                                                              BlockT *Node) {
   assert(I != end() && "BB is not in DominanceFrontier!");
-  assert(I->second.count(Node) && "Node is not in DominanceFrontier of BB");
-  I->second.erase(Node);
+  I->second.insert(Node);
 }
 
 template <class BlockT, bool IsPostDom>


        


More information about the llvm-commits mailing list