summaryrefslogtreecommitdiffstats
path: root/llvm/lib
diff options
context:
space:
mode:
authorNirav Dave <niravd@google.com>2017-08-10 21:03:36 +0000
committerNirav Dave <niravd@google.com>2017-08-10 21:03:36 +0000
commitf8556ad48f5ba9d020509ea7833cae432730bb19 (patch)
tree9b4c457a0bf73ce5bd9a58355a1b8c6de21f443b /llvm/lib
parent9a6110b2d3ea6c037923aff08c21e047999c5a47 (diff)
downloadbcm5719-llvm-f8556ad48f5ba9d020509ea7833cae432730bb19.tar.gz
bcm5719-llvm-f8556ad48f5ba9d020509ea7833cae432730bb19.zip
Revert "[DAG] Cleanup unused nodes after store merge. NFCI."
This reverts commit r310648 which causes an unexpected assertion failure llvm-svn: 310659
Diffstat (limited to 'llvm/lib')
-rw-r--r--llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp12
1 files changed, 1 insertions, 11 deletions
diff --git a/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp b/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
index 158350dc6d2..7515a7ab6d7 100644
--- a/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
+++ b/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
@@ -12590,15 +12590,8 @@ bool DAGCombiner::MergeStoresOfConstantsOrVecElts(
}
// Replace all merged stores with the new store.
- for (unsigned i = 0; i < NumStores; ++i) {
- SDValue Val = StoreNodes[i].MemNode->getOperand(1);
- SDValue Addr = StoreNodes[i].MemNode->getOperand(2);
+ for (unsigned i = 0; i < NumStores; ++i)
CombineTo(StoreNodes[i].MemNode, NewStore);
- if (Val.getNode()->use_empty())
- recursivelyDeleteUnusedNodes(Val.getNode());
- if (Addr.getNode()->use_empty())
- recursivelyDeleteUnusedNodes(Addr.getNode());
- }
AddToWorklist(NewChain.getNode());
return true;
@@ -13233,12 +13226,9 @@ bool DAGCombiner::MergeConsecutiveStores(StoreSDNode *St) {
// corresponding value if its no longer used.
for (unsigned i = 0; i < NumElem; ++i) {
SDValue Val = StoreNodes[i].MemNode->getOperand(1);
- SDValue Addr = StoreNodes[i].MemNode->getOperand(2);
CombineTo(StoreNodes[i].MemNode, NewStore);
if (Val.getNode()->use_empty())
recursivelyDeleteUnusedNodes(Val.getNode());
- if (Addr.getNode()->use_empty())
- recursivelyDeleteUnusedNodes(Addr.getNode());
}
RV = true;
OpenPOWER on IntegriCloud