[FFmpeg-devel] [PATCH] SPARC VIS simple_idct
Michael Niedermayer
michaelni
Sat Aug 25 21:01:00 CEST 2007
Hi
On Fri, Aug 24, 2007 at 11:02:29PM -0700, Michel Lespinasse wrote:
> On Sat, Aug 25, 2007 at 01:20:45AM +0200, Balatoni Denes wrote:
>
> > ps: there is a half as fast version of this idct, but that's
> > accurate (32 bit multiplies) - I am wondering if maybe that would
> > make more sense in ffmpeg. Or there could be three sparc idcts: one
> > slow (but faster than the C version) and accurate, one faster but
> > less accurate, and then the mlib (fastest, very inaccurate, mpeg4
> > routinely turns pink while viewing). Maybe it's not a good idea.
>
> One can make an accurate enough IDCT using VIS, the 8x16 bit multiplies
> make that very hard but not quite entirely impossible.
>
> I finally dug out the old test code I had written in september 2003,
> at the time david miller was interested to convert that to asm and then
> he got busy with other things. It works on columns, then transposes
> the table and does another pass. David thought he could make it faster
> than the VIS one, but hey, talk is cheap :) The test code is in C but
> I believe the muls/mulu functions match what VIS implements. idct() is
> what you'd want to be fast, idct_vis() is the C function I used to hook
> this into my IEEE1180 test framework, it reorders input coefficients
> and preshifts them by 4 (libmpeg2 prescales IDCT during stream parsing).
>
> Don't know if there is any interest or how it compares with the simple-idct
> derived code - but, it is (barely) IEEE1180 compliant and does not use
> 32-bit multiplies.
>
> Hope this helps,
of course it does
btw you dont maybe also have that in asm() form?
btw2, if you ever want a ffmpeg svn write account ... just ask
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
In a rich man's house there is no place to spit but his face.
-- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070825/16c11098/attachment.pgp>
More information about the ffmpeg-devel
mailing list