[Fwd: Re: libtheora MMX patch]

Forwarding an email exchange that I had with Nils Pipenbrinck regarding 
the state of the MMX patch for visual studio-style assembly.

I also run with the patches, and everything looks fine as far as I can tell.

Is this enough for a go-ahead to put that stuff into the mainline (if 
it's not there already)?


-------- Original Message --------
Subject: 	Re: libtheora MMX patch
Date: 	Sun, 06 Apr 2008 15:21:12 +0200
From: 	Nils Pipenbrinck <[email protected]>
To: 	Andrew Chew <[email protected]>
References: 	<[email protected]> 
<[email protected]> <[email protected]> 
<[email protected]>



Hi Andrew,

I run the original C-code against the mmx optimized version, dumped out 
the raw yuv-frames to disk and did a binary compare. It's functional 
equivalent down to the bit-level. I used two different test-streams for 
this. It's highly unlikely that a bug survives this test.

AFAIK my changes for the gcc inline assembler functions are already in 
the trunk (slightly modified - Giles didn't liked the fact that I 
unrolled / wrote the loops in asm).

If you want to merge into the win32 baseline let me know I have adopted 
the filter and idct mmx functions to match more the gcc style but never 
submitted them. Also a little bug crept into the win32 mmx code you 
have. I accidently used the same code for idct10 and idct. Functional it 
works but I lost a little bit speed that way (barely measurable but 
well...).

I have a local version where this is fixed as well.

If you're interested I'll dig out the modified codes from my old 
harddisk and send it to you.

 Nils







Andrew Chew schrieb:
> Hi!  I tested your assembly patches to libtheora a while back.
>
> How well have you validated that it works?  It works in my environment 
> pretty well.
>
> I'd just like to make sure this gets into mainline libtheora, and so 
> far it hasn't (they are waiting for some confirmation that those 
> patches are indeed good).  I guess they are short on WIN32 testers.  
> This was revealed to me at the last libtheora roadmap meeting that we 
> just had last week.
>
> I'd really like to make sure that your stuff gets into their mainline.