[PATCH] D127115: [RFC][DAGCombine] Make sure combined nodes are added back to the worklist in topological order.

[Amaury SECHET via Phabricator via llvm-commits]

deadalnix added a comment.

In D127115#3674147 <https://reviews.llvm.org/D127115#3674147>, @barannikov88 wrote:

> @deadalnix 
> Could you please answer this question? https://reviews.llvm.org/D127115/new/#3668015
>
>> Does the topological order here mean that the DAG will be traversed from the root to leaves (RPO) or from the leaves to the root (PO)?



- The code answer that question.
- The diff's description answer that question.
- Several comments in the thread already answer that question.

It'll visit arguments before the node that use arguments. It's the whole point.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127115