[cfe-commits] r141053 - in /cfe/trunk: include/clang/Basic/DiagnosticDriverKinds.td lib/Driver/Tools.cpp
kremenek at apple.com
Tue Oct 4 09:52:25 PDT 2011
On Oct 4, 2011, at 9:03 AM, Bob Wilson wrote:
> I'm not sure this is a good idea. I understand the concern about having -W flags for all warnings, but we certainly don't anyone to start using -Wno-unsupported-debug-with-integrated-as.
Can you clarify your concern? If people know what they are doing, they should be able to silence the warning (or promote them to an error).
> As soon as the integrated assembler can support debug info, we're going to rip out this warning, and we don't want to be stuck keeping the -W flags around for backward compatibility.
I don't see that as an issue. We never promise that warnings will stay around forever. Once a warning is deemed not to be useful anymore we should feel free to remove it.
> Is there some existing warning group that we could use for this?
No, and I don't think that would be the right approach anyway. That seems counter to your concern about putting this warning under a flag in the first place.
If you really don't want driver warnings to be under a -W flag, then we should consider adding another diagnostic category to segregate them from other warnings. Once we get all warnings under a -W flag, we'd like to change tablegen to reject warnings that are not in a diagnostic group.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits