On Tue, Oct 23, 2012 at 01:34:41PM +0200, Diego Biurrun wrote:
> --- a/libavutil/lzo.c
> +++ b/libavutil/lzo.c
> @@ -145,16 +133,16 @@ static inline void memcpy_backptr(uint8_t *dst, int
> back, int cnt)
> memset(dst, *src, cnt);
> } else {
> #ifdef OUTBUF_PADDED
> - COPY2(dst, src);
> - COPY2(dst + 2, src + 2);
> + AV_COPY16U(dst, src);
> + AV_COPY16U(dst + 2, src + 2);
> src += 4;
> dst += 4;
> cnt -= 4;
> if (cnt > 0) {
> - COPY2(dst, src);
> - COPY2(dst + 2, src + 2);
> - COPY2(dst + 4, src + 4);
> - COPY2(dst + 6, src + 6);
> + AV_COPY16U(dst, src);
> + AV_COPY16U(dst + 2, src + 2);
> + AV_COPY16U(dst + 4, src + 4);
> + AV_COPY16U(dst + 6, src + 6);
> src += 8;
> dst += 8;
> cnt -= 8;
For some reason that I do not understand coalescing the AV_COPY16U calls
to AV_COPY32U calls breaks the dfa decoder fate test.
Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
