diff options
| author | Mike Frysinger <vapier@gentoo.org> | 2012-07-31 09:02:35 +0000 |
|---|---|---|
| committer | Mike Frysinger <vapier@gentoo.org> | 2012-07-31 09:02:35 +0000 |
| commit | 902e4335501e73ad45ad2ebc2510b1d41786873f (patch) | |
| tree | 2699b97dd537d8bbf8605c398d4a18106bb55469 /libiberty/ChangeLog | |
| parent | 4a2608e354d22289f40944bb29d85db3f3c5ca04 (diff) | |
| download | gdb-902e4335501e73ad45ad2ebc2510b1d41786873f.zip gdb-902e4335501e73ad45ad2ebc2510b1d41786873f.tar.gz gdb-902e4335501e73ad45ad2ebc2510b1d41786873f.tar.bz2 | |
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.
Diffstat (limited to 'libiberty/ChangeLog')
| -rw-r--r-- | libiberty/ChangeLog | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/libiberty/ChangeLog b/libiberty/ChangeLog index 349dc98..6615047 100644 --- a/libiberty/ChangeLog +++ b/libiberty/ChangeLog @@ -1,3 +1,8 @@ +2012-07-31 Mike Frysinger <vapier@gentoo.org> + + * md5.c (md5_finish_ctx): Declare swap_bytes. Assign SWAP() output + to swap_bytes, and then call memcpy to move it to ctx->buffer. + 2012-07-26 Kazu Hirata <kazu@codesourcery.com> Sandra Loosemore <sandra@codesourcery.com> |