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

2011-04-30 Thread Uoti Urpala
On Tue, 2011-04-26 at 10:42 +0100, Måns Rullgård wrote: > Uoti Urpala 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 > >> wrote: > >> > The current generic C implementation, which is always used when the > >> > public head

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

2011-04-26 Thread Måns Rullgård
Uoti Urpala 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 wrote: >> > The current generic C implementation, which is always used when the >> > public header is included from other programs (either directly or >> > through int

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

2011-04-24 Thread Alexander Strange
On Sun, Apr 24, 2011 at 9:18 AM, Clément Bœsch wrote: > On Sun, Apr 24, 2011 at 09:06:16AM +0300, Uoti Urpala 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

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 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 m

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

2011-04-24 Thread Clément Bœsch
On Sun, Apr 24, 2011 at 09:06:16AM +0300, Uoti Urpala 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): > movq%rdi, %

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 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): >        movq    %rdi, %rdx

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

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

2011-04-23 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 sall