[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