[llvm] [DAGCombiner] cache negative result from getMergeStoreCandidates() (PR #106949)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 3 23:34:41 PDT 2024
================
@@ -191,6 +191,11 @@ namespace {
// AA - Used for DAG load/store alias analysis.
AliasAnalysis *AA;
+ /// This caches all chains that have already been processed in
+ /// DAGCombiner::getStoreMergeCandidates() and found to have no mergeable
+ /// stores candidates.
+ SmallSet<SDNode *, 32> ChainsWithoutMergeableStores;
----------------
arsenm wrote:
I didn't know it had the specialization, last I looked it didn't. I guess I still expect to see SmallPtrSet. These also are supposed to have a default value, so you should just use that unless you have a specific reason to pick another number
https://github.com/llvm/llvm-project/pull/106949
More information about the llvm-commits
mailing list