Displaying 1 result from an estimated 1 matches for "storenodes".
Did you mean:
storesdnodes
2020 Mar 19
2
large slowdown in DAGCombiner::MergeConsecutiveStores
...ng very long chains of stores. For example, this change makes the
problem go away:
```
--- a/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
+++ b/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
@@ -16011,6 +16011,9 @@ bool
DAGCombiner::MergeConsecutiveStores(StoreSDNode *St) {
StoreNodes.begin() + NumConsecutiveStores);
continue;
}
+ else if (NumConsecutiveStores > 128) {
+ return RV;
+ }
```
Does anybody have any advice on what a good way to fix this would be? Does
it make sense to have some kind of limit on how many stores we try to
merge? (Even if so,...