[PATCH] D70921: [DAGCombiner] Add node back in the worklist in topological order in CommitTargetLoweringOpt

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 9 12:25:44 PST 2019


lebedev.ri accepted this revision.
lebedev.ri added a comment.
This revision is now accepted and ready to land.

... where `AddToWorklistWithUsers()` calls those two functions in the opposite order.
Looks ok to me. Might be good to have a test case, but i don't expect that it would be easy to produce one.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70921/new/

https://reviews.llvm.org/D70921





More information about the llvm-commits mailing list