[FFmpeg-devel] [PATCH] fix for roundup issue 2127

Daniel Kang daniel.d.kang
Thu Dec 30 19:38:24 CET 2010


Sorry, I sent the wrong patch (the one I used for preliminary testing). Here
is the real version.

On Thu, Dec 30, 2010 at 1:33 PM, Reimar D?ffinger
<Reimar.Doeffinger at gmx.de>wrote:

> On 30 dec 2010, at 19:14, Daniel Kang <daniel.d.kang at gmail.com> wrote:
>
> > I am a Google Code-In student, and as part of a task, I have written a
> > fix for: https://roundup.ffmpeg.org/issue2127. The patch compiles with
> > 32/64-bit and --disable-optimizations. It passes "make fate" with and
> > without --disable-optimizations.
> >
> > I have ran several timings with and without my patch. The
> > START/STOP_TIMER was placed around the actual "asm" block in the
> > transpose4x4 function.
> > With my patch: 21.32 +/- 0.04 clocks
> > Without my patch: 21.58 +/- 0.04 clocks
> >
> > Are there any comments on my patch? If not, can this be committed?
>
> There must be something wrong, the #else code will never be compiled.
> Also, the new code seems to require 6 registers, IIRC PIC + frame-pointer
> only leaves available 5.
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix.diff
Type: application/octet-stream
Size: 2599 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101230/8306fc34/attachment.obj>



More information about the ffmpeg-devel mailing list