On Jun 18, 2011, at 12:00 PM, Richard Osborne wrote:
> This patch adds support for printing diagnostic pragmas to the preprocessed
> output. Previously they would be discarded
> (http://llvm.org/bugs/show_bug.cgi?id=5952).
>
> Please let me know if this patch looks OK.
Your patch looks very good.
> Note diagnostic pragmas in the clang and GCC pragma namespaces will both be
> printed as "#pragma GCC diagnostic ...". I assumed this wouldn't matter as
> they should have the same behaviour.
I'd much prefer that we get the output perfect, by tracking which pragma
namespace was specified and emitting the output the same way. Otherwise, we'll
just end up getting a bug report 6 months down the line asking for more precise
output :)
- Doug
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
