[llvm] faf555f - Revert "[DAGCombiner] Use SmallDenseMap (NFC) (#79681)"

Kazu Hirata via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 27 19:29:53 PST 2024


Author: Kazu Hirata
Date: 2024-01-27T19:29:47-08:00
New Revision: faf555f93f3628b7b2b64162c02dd1474540532e

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

LOG: Revert "[DAGCombiner] Use SmallDenseMap (NFC) (#79681)"

This reverts commit 863b2c84c0fbcfb02d969fa36af4932d410a827b.

A compile-time regression has been reported:

https://github.com/llvm/llvm-project/pull/79681#issuecomment-1913325915

Added: 
    

Modified: 
    llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp b/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
index ab572c5eca73998..87184fe409eade6 100644
--- a/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
+++ b/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
@@ -174,7 +174,7 @@ namespace {
     /// This is used to find and remove nodes from the worklist (by nulling
     /// them) when they are deleted from the underlying DAG. It relies on
     /// stable indices of nodes within the worklist.
-    SmallDenseMap<SDNode *, unsigned, 32> WorklistMap;
+    DenseMap<SDNode *, unsigned> WorklistMap;
 
     /// This records all nodes attempted to be added to the worklist since we
     /// considered a new worklist entry. As we keep do not add duplicate nodes


        


More information about the llvm-commits mailing list