[PATCH] D70921: [DAGCombiner] Add node back in the worklist in topological order in CommitTargetLoweringOpt
Amaury SECHET via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 9 07:00:09 PST 2019
deadalnix updated this revision to Diff 232843.
deadalnix added a comment.
Ping, rebase
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70921/new/
https://reviews.llvm.org/D70921
Files:
llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
Index: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
===================================================================
--- llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
+++ llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
@@ -1035,8 +1035,7 @@
DAG.ReplaceAllUsesOfValueWith(TLO.Old, TLO.New);
// Push the new node and any (possibly new) users onto the worklist.
- AddToWorklist(TLO.New.getNode());
- AddUsersToWorklist(TLO.New.getNode());
+ AddToWorklistWithUsers(TLO.New.getNode());
// Finally, if the node is now dead, remove it from the graph. The node
// may not be dead if the replacement process recursively simplified to
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D70921.232843.patch
Type: text/x-patch
Size: 647 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191209/719f5893/attachment.bin>
More information about the llvm-commits
mailing list