This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PATCH: improve ATTRIBUTE_GCC_DIAG


On Thu, May 7, 2009 at 9:59 PM, Ben Elliston <bje@au1.ibm.com> wrote:
>> Maybe it is better to only enable the warning if the version we are
>> compiling is the same as it is being compiled with.
>
> That means you would only discover format string problems when
> bootstrapping (unless you keep a build compiler around of the same
> version).
>
> For those developers working in cross environments, they would also have
> to keep a native compiler of the same version installed to see the
> warnings. ÂI think that is a poorer proposition, but that's just my 2
> cents.

Except if you look at the original problem is that the warnings are
not correct for when the two compilers are not the same version.  So
disabling the warnings for !ENABLE_CHECKING just means you don't get
the warnings on the release branches but do get them for the trunk (if
the two compilers are not the same version) which could confuse people
even more than my proposal.

Thanks,
Andrew Pinski


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]