Re: [FFmpeg-devel] [PATCH 10/11] avcodec/blockdsp: add AVX-512 version of clear_block(s)

2017-11-10 Thread James Almer
On 11/10/2017 10:28 AM, James Darnley wrote: > On 2017-11-09 20:35, Martin Vignali wrote: >> 2017-11-09 12:58 GMT+01:00 James Darnley : >> >>> From: James Darnley >>> >>> Also adjust alignment requirements where nessecary. >>> --- >>> Whether this patch

Re: [FFmpeg-devel] [PATCH 10/11] avcodec/blockdsp: add AVX-512 version of clear_block(s)

2017-11-10 Thread James Darnley
On 2017-11-09 20:35, Martin Vignali wrote: > 2017-11-09 12:58 GMT+01:00 James Darnley : > >> From: James Darnley >> >> Also adjust alignment requirements where nessecary. >> --- >> Whether this patch is committed or not the change to 4xm.c should be >>

Re: [FFmpeg-devel] [PATCH 10/11] avcodec/blockdsp: add AVX-512 version of clear_block(s)

2017-11-09 Thread Martin Vignali
2017-11-09 12:58 GMT+01:00 James Darnley : > From: James Darnley > > Also adjust alignment requirements where nessecary. > --- > Whether this patch is committed or not the change to 4xm.c should be > picked to > master because the alignment is wrong for

[FFmpeg-devel] [PATCH 10/11] avcodec/blockdsp: add AVX-512 version of clear_block(s)

2017-11-09 Thread James Darnley
From: James Darnley Also adjust alignment requirements where nessecary. --- Whether this patch is committed or not the change to 4xm.c should be picked to master because the alignment is wrong for the AVX version of this function. I assume it hasn't been noticed yet