[PATCH] D35175: New option that adds the DiagID enum name and index to Diagnostic output.

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 11 19:36:49 PDT 2017


rjmccall added a comment.

This is a cute hack, but... I'm not sure I accept the premise that it's a noteworthy obstacle to Clang development to do two greps instead of one.  And I don't think I've ever had to debug a diagnostic where __FILE__ and __LINE__ information would've been helpful.

Also, you could easily write a script that could automatically annotate arbitrary text with this information retroactively by turning the diagnostic text into a bunch of regular expressions, and then you wouldn't even need to re-run the compiler.


https://reviews.llvm.org/D35175





More information about the cfe-commits mailing list