Re: [FFmpeg-devel] [PATCH 3/4] hevc: move intermediate bidir buffer

2014-09-23 Thread Michael Niedermayer
On Tue, Sep 23, 2014 at 09:36:42PM +, Christophe Gisquet wrote: > Other buffers are already there. > --- > libavcodec/hevc.c | 14 ++ > libavcodec/hevc.h | 1 + > 2 files changed, 7 insertions(+), 8 deletions(-) aligned stack variables can cause issues in some compilers IIRC so

Re: [FFmpeg-devel] [PATCH 3/4] hevc: move intermediate bidir buffer

2014-09-23 Thread Christophe Gisquet
2014-09-23 23:36 GMT+02:00 Christophe Gisquet : > Other buffers are already there. Note: if this is not applied, I think tmp should use some variant of LOCAL_ALIGNED_16 (or 32 once avx2 gets here). The edge_emu_buffers are big (more than 22K), but I'm not sure what the benefit is over having them

[FFmpeg-devel] [PATCH 3/4] hevc: move intermediate bidir buffer

2014-09-23 Thread Christophe Gisquet
Other buffers are already there. --- libavcodec/hevc.c | 14 ++ libavcodec/hevc.h | 1 + 2 files changed, 7 insertions(+), 8 deletions(-) diff --git a/libavcodec/hevc.c b/libavcodec/hevc.c index 4e9948f..4860aec 100644 --- a/libavcodec/hevc.c +++ b/libavcodec/hevc.c @@ -1328,7 +1328,