[PATCH] D23730: [GraphTraits] Replace all NodeType usage with NodeRef

Tim Shen via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 22 12:58:39 PDT 2016


timshen added a comment.

In https://reviews.llvm.org/D23730#522396, @dblaikie wrote:

> Not all of these already had NodeRef implemented - that implies that some algorithms weren't using NodeRef before this change, or that these traits are unused? I thought the plan was to migrate each algorithm then just do a strict cleanup. Did that not pan out/some other aspects I'm forgetting?


I think those traits are unused. Notice that by merely removing "NodeType *" declarations, the build/tests doesn't break, so at least they are not covered by the tests.


https://reviews.llvm.org/D23730





More information about the llvm-commits mailing list