Diego Biurrun <di...@biurrun.de> writes:
> 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.
The offset is less than 4.
--
Måns Rullgård
m...@mansr.com
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
