[theora-dev] [Fwd: Re: libtheora MMX patch]

[Andrew Chew]

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 <n.pipenbrinck at cubic.org>
To: 	Andrew Chew <andrew at okashistudios.com>
References: 	<4778152E.5010904 at cubic.org> 
<477857B8.1010102 at email.unc.edu> <4778AA4C.9010105 at cubic.org> 
<47F8463A.8000007 at okashistudios.com>



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.