I'm not sure why "replace" is the right interface. Wouldn't it be better to 
just have an AnalysisDiagClients entry for "no default 
PathDiagnosticConsumers", and then be able to add to them?

  I'm also still not convinced that you really need a //new// 
PathDiagnosticConsumer, rather than, say, improving the text diagnostics.

http://llvm-reviews.chandlerc.com/D2556
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to