Re: [libav-devel] [PATCH] Use GCC builtins in bswap.h

2011-04-26 Thread Måns Rullgård
Uoti Urpala uoti.urp...@pp1.inet.fi writes: On Sun, 2011-04-24 at 07:05 -0400, Ronald S. Bultje wrote: On Sun, Apr 24, 2011 at 2:06 AM, Uoti Urpala uoti.urp...@pp1.inet.fi wrote: The current generic C implementation, which is always used when the public header is included from other

[libav-devel] [PATCH] Use GCC builtins in bswap.h

2011-04-24 Thread Uoti Urpala
The current generic C implementation, which is always used when the public header is included from other programs (either directly or through intreadwrite.h) compiles to a mess like this (gcc-4.6 on AMD64): movq%rdi, %rdx shrq$32, %rdx movl%edx, %eax

Re: [libav-devel] [PATCH] Use GCC builtins in bswap.h

2011-04-24 Thread Luca Barbato
On 4/24/11 8:06 AM, Uoti Urpala wrote: +#elif 0 Maybe if 0 code could be removed or investigated. Beside that the patch seems ok. lu ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] Use GCC builtins in bswap.h

2011-04-24 Thread Ronald S. Bultje
Hi, On Sun, Apr 24, 2011 at 2:06 AM, Uoti Urpala uoti.urp...@pp1.inet.fi wrote: The current generic C implementation, which is always used when the public header is included from other programs (either directly or through intreadwrite.h) compiles to a mess like this (gcc-4.6 on AMD64):        

Re: [libav-devel] [PATCH] Use GCC builtins in bswap.h

2011-04-24 Thread Uoti Urpala
On Sun, 2011-04-24 at 07:05 -0400, Ronald S. Bultje wrote: On Sun, Apr 24, 2011 at 2:06 AM, Uoti Urpala uoti.urp...@pp1.inet.fi wrote: The current generic C implementation, which is always used when the public header is included from other programs (either directly or through