[FFmpeg-devel] [PATCH] Fix non-rounding up to next 16-bit aligned bug in IFF decoder

[Ronald S. Bultje]

Hi,

On Thu, Apr 29, 2010 at 2:22 PM, Sebastian Vater
<cdgs.basty at googlemail.com> wrote:
> Regarding your idea gcc outputs:
> ? ? ?52: ? ? ? 0f b6 16 ? ? ? ? ? ? ? ?movzbl (%esi),%edx
> ? ? ?55: ? ? ? 83 c6 01 ? ? ? ? ? ? ? ?add ? ?$0x1,%esi
> ? ? ?58: ? ? ? 89 d0 ? ? ? ? ? ? ? ? ? mov ? ?%edx,%eax
> ? ? ?5a: ? ? ? 83 e2 0f ? ? ? ? ? ? ? ?and ? ?$0xf,%edx
> ? ? ?5d: ? ? ? c1 e8 04 ? ? ? ? ? ? ? ?shr ? ?$0x4,%eax
> ? ? ?60: ? ? ? 8b 04 81 ? ? ? ? ? ? ? ?mov ? ?(%ecx,%eax,4),%eax
> ? ? ?63: ? ? ? 09 03 ? ? ? ? ? ? ? ? ? or ? ? %eax,(%ebx)
> ? ? ?65: ? ? ? 8b 04 91 ? ? ? ? ? ? ? ?mov ? ?(%ecx,%edx,4),%eax
> ? ? ?68: ? ? ? 09 43 04 ? ? ? ? ? ? ? ?or ? ? %eax,0x4(%ebx)
> ? ? ?6b: ? ? ? 83 c3 08 ? ? ? ? ? ? ? ?add ? ?$0x8,%ebx
> ? ? ?6e: ? ? ? 39 df ? ? ? ? ? ? ? ? ? cmp ? ?%ebx,%edi
> ? ? ?70: ? ? ? 77 e0 ? ? ? ? ? ? ? ? ? ja ? ? 52 <decodeplane8+0x52>

12 instructions, so 2 less, as intended. Is it faster?

Ronald