[Ffmpeg-devel] [RFC] port cabac asm to AMD64

[Michael Niedermayer]

Hi

On Fri, Oct 20, 2006 at 11:08:46PM +0200, Reimar D?ffinger wrote:
> Hello,
> attached (admittedly a bit hackish but minimal) patch allows to use the
> cabac asm optimizations to be used on AMD64.
> It may be a bit early to do this, but I wondered if maybe someone would
> be less lazy than me and do some benchmarking.
> Also, I wondered if there is a reason to use the "l" prefix in the asm,

no reason


> because I have to remove it in may places to make it compile, and all
> the code ported to AMD64 AFAICT had it removed. Would a patch to remove
> them despite the early state of the optimizations be accepted?

yes


> 
> Greetings,
> Reimar D?ffinger

[...]
> -#if defined(ARCH_X86) && !(defined(PIC) && defined(__GNUC__))
> +#endif
> +#if defined(ARCH_X86)&& !(defined(PIC) && defined(__GNUC__))

do i see a cosmetic change here ?!

except that patch ok, assuming it still works with current svn

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is