ayartsev added inline comments.

================
Comment at: test/Analysis/PR12421.c:11
@@ +10,2 @@
+
+// CHECK: warning: Path diagnostic report is not generated. HTMLDiagnostics 
does not support diagnostics that cross file boundaries.
----------------
zaks.anna wrote:
> We should use the name of the diagnostic consumer here - that will only be 
> legible for the developers working on the attic analyzer core. 
Done. As for me the name of the diagnostic consumer also makes the warning more 
clear and helpful for an ordinary user. From the consumer name he can see what 
report format is talked about and maybe change the scan-build (which setups the 
'-analyzer-output' frontend option internally) options. Do you still want to 
remove the consumer name from the warning?


https://reviews.llvm.org/D22494



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

Reply via email to