diff options
author | Rich Felker <dalias@aerifal.cx> | 2011-04-12 13:13:27 -0400 |
---|---|---|
committer | Rich Felker <dalias@aerifal.cx> | 2011-04-12 13:13:27 -0400 |
commit | c89862660bd3845ace7977480db3a43dc80475f4 (patch) | |
tree | 4b3c890588e359f7e7674898acecbe1aacd95336 /src/network/ntohl.c | |
parent | c546be175c169399420fff4706cd1d4fcefd7d31 (diff) | |
download | musl-c89862660bd3845ace7977480db3a43dc80475f4.tar.gz musl-c89862660bd3845ace7977480db3a43dc80475f4.tar.bz2 musl-c89862660bd3845ace7977480db3a43dc80475f4.tar.xz musl-c89862660bd3845ace7977480db3a43dc80475f4.zip |
optimize ntohl etc. in terms of bswap functions
we can do this without violating the namespace now that they are
macros/inline functions rather than extern functions. the motivation
is that gcc was generating giant, slow, horrible code for the old
functions, and now generates a single byte-swapping instruction.
Diffstat (limited to 'src/network/ntohl.c')
-rw-r--r-- | src/network/ntohl.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/src/network/ntohl.c b/src/network/ntohl.c index 64379196..d6fce459 100644 --- a/src/network/ntohl.c +++ b/src/network/ntohl.c @@ -1,10 +1,8 @@ #include <netinet/in.h> +#include <byteswap.h> uint32_t ntohl(uint32_t n) { - union { - uint32_t i; - uint8_t b[4]; - } u = { n }; - return (u.b[0]<<24) | (u.b[1]<<16) | (u.b[2]<<8) | u.b[3]; + union { int i; char c; } u = { 1 }; + return u.c ? bswap_32(n) : n; } |