dcoughlin added a comment.

This is seems like a very useful visualization, *especially* for loops. Can we 
this patch get it into a state where it can be committed and used for debugging 
purposes?

One possibility is to turn this into a debug checker similar to 
debug.ViewExplodedGraph. That checker registers for a checkEndAnalysis() 
callback and traverses the node graph (see DebugCheckers.cpp). Can you do the 
same here? It doesn't look like you really need this to be a BugReporterVisitor 
-- and making it a debug checker would avoid outputting multiple copies for 
each diagnostic consumer.

You could also control file output with an analyzer-config option that takes an 
optional path.


https://reviews.llvm.org/D40809



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to