checking changes for warning regressions
H.J. Lu
hjl.tools@gmail.com
Wed Oct 3 00:09:00 GMT 2012
On Tue, Oct 2, 2012 at 4:23 PM, Roland McGrath <roland@hack.frob.com> wrote:
> Your commit ae30640a of 2012-9-19 has two issues.
>
> First, you didn't update/collapse copyright years.
Fixed.
> Second, this change introduced several -Wpointer-to-int-cast warnings.
>
> Everybody, please consider it a responsibility for any change you do to
> try the compilation on some relevant configuration and verify that it
> does not introduce new warnings into the build with some fairly recent
> compiler (I'm using 4.6.3).
>
> In this case, the warning doesn't indicate any possible bug. It's in an
> if branch that will always be eliminated dead code in any circumstance
> where the warning is issued. But it's still bad to introduce such warnings.
> (I don't think I need to elaborate on why it's bad. I can if necessary.)
>
> I haven't thought of a way to eliminate this warning without losing the
> functional aspect of your change. But I haven't put much effort into
> finding one. Please see if you can find one.
>
The warnings have always been there for x32 build. I investigated it
while working on x32 and came to the same conclusion. This is the
known problem:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=4210
--
H.J.
More information about the Libc-alpha
mailing list