[PATCH] D34295: [Dominators] Move number to node mapping out of DominatorTreeBase

Daniel Berlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 16 21:58:14 PDT 2017


dberlin added a comment.

This is fine, but at the end of all these patches, can you please make it so there is only one structure storing state for building?

(i don't care whether this is by making genericdomtreebuilder a real class, or passing a single struct around)
I'm okay if the SNCAInfo is a pointer in that struct :)


Repository:
  rL LLVM

https://reviews.llvm.org/D34295





More information about the llvm-commits mailing list