diff mbox

[FFmpeg-devel,2/2] dstdec: use appropriate alignment

Message ID bc1da5b3fa8df78dc051d8e2f87071b2157e9d8a.1545359356.git.pross@xvid.org
State Accepted
Commit ad0d5d7516dc3de7d1172c03920256b6a48fcd39
Headers show

Commit Message

Peter Ross Dec. 21, 2018, 2:34 a.m. UTC 
this was a typo in my original patch. there is no requirement for 64-byte
alignment here, and it does not affect decoder performance.
---
 libavcodec/dstdec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Michael Niedermayer Dec. 22, 2018, 8:38 a.m. UTC | #1 
On Fri, Dec 21, 2018 at 01:34:19PM +1100, Peter Ross wrote:
> this was a typo in my original patch. there is no requirement for 64-byte
> alignment here, and it does not affect decoder performance.
> ---
>  libavcodec/dstdec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

LGTM

thx

[...]
diff mbox

Patch

diff --git a/libavcodec/dstdec.c b/libavcodec/dstdec.c
index 368cb64931..511861f4d2 100644
--- a/libavcodec/dstdec.c
+++ b/libavcodec/dstdec.c
@@ -70,7 +70,7 @@  typedef struct DSTContext {
     GetBitContext gb;
     ArithCoder ac;
     Table fsets, probs;
-    DECLARE_ALIGNED(64, uint8_t, status)[DST_MAX_CHANNELS][16];
+    DECLARE_ALIGNED(16, uint8_t, status)[DST_MAX_CHANNELS][16];
     DECLARE_ALIGNED(16, int16_t, filter)[DST_MAX_ELEMENTS][16][256];
     DSDContext dsdctx[DST_MAX_CHANNELS];
 } DSTContext;