[PATCH] D50479: Expose CFG Update struct. Define GraphTraits to get children given a snapshot CFG.

Jakub Kuderski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 9 13:07:35 PDT 2018


kuhar added a comment.

In https://reviews.llvm.org/D50479#1194111, @asbirlea wrote:

> I agree that long term this can and should be reused in DomTree, so I'm trying to make it have the right functionality.
>  Another possibly related issue, is that I'm not reversing the children, as in DomTree, not sure the reason behind that one.


Sure. The children are being reversed to preserve the successor order in DFS, which uses stack and reverses them (again) internally. But this reverse can be moved somewhere else and you don't have to worry about it.


Repository:
  rL LLVM

https://reviews.llvm.org/D50479





More information about the llvm-commits mailing list