This is the mail archive of the gdb@sourceware.org mailing list for the GDB 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: New GDB 7.5.x release sometime next week?


> Subject: [PATCH] libiberty/md5: fix strict alias warnings
> 
> Current libiberty md5 code triggers these warnings with gcc-4.7.1 for me:
> 
> libiberty/md5.c: In function 'md5_finish_ctx':
> libiberty/md5.c:117:3: warning: dereferencing type-punned pointer will break 
> strict-aliasing rules [-Wstrict-aliasing]
> libiberty/md5.c:118:3: warning: dereferencing type-punned pointer will break 
> strict-aliasing rules [-Wstrict-aliasing]
> 
> The change below fixes things for me.  The optimized output (-O2) is
> the same before/after my change on x86_64-linux.  I imagine it'll be
> the same for most targets.  It seems simpler than using a union on the
> md5_ctx buffer since these are the only two locations in the code
> where this occurs.

I am on the fence regarding this change. On the one hand, it seems
hardly critical, since it only fixes some warnings, and does not seem
to cause any improvement in the actual code generated. On the other
hand, this module appears to be completely unused in all of GDB (?).
So the risk would be small. For that reason, and the fact that the code
has been in since end of July, I think it's OK to include it, and
avoid the warnings.

Thanks,
-- 
Joel


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