[FFmpeg-devel] [PATCH] Make DECLARE_ALIGNED_8 align to 8 bytes, never to 16

Mans Rullgard mans
Thu Aug 27 00:50:21 CEST 2009


The DECLARE_ALIGNED_8 macro is defined to align to 16 bytes instead
the 8 suggested by the name on some CPUs.  None of the uses of this
macro ever need 16-byte alignment, cases which once did having been
changed to always specify 16 bytes explicitly.
---
 libavcodec/dsputil.h |    5 -----
 1 files changed, 0 insertions(+), 5 deletions(-)

diff --git a/libavcodec/dsputil.h b/libavcodec/dsputil.h
index 2d15bd3..1c3a82b 100644
--- a/libavcodec/dsputil.h
+++ b/libavcodec/dsputil.h
@@ -639,7 +639,6 @@ void dsputil_init_pix_mmx(DSPContext* c, AVCodecContext *avctx);
 extern int mm_flags;
 
 #if HAVE_NEON
-#   define DECLARE_ALIGNED_8(t, v) DECLARE_ALIGNED(16, t, v)
 #   define STRIDE_ALIGN 16
 #endif
 
@@ -647,12 +646,10 @@ extern int mm_flags;
 
 extern int mm_flags;
 
-#define DECLARE_ALIGNED_8(t, v) DECLARE_ALIGNED(16, t, v)
 #define STRIDE_ALIGN 16
 
 #elif HAVE_MMI
 
-#define DECLARE_ALIGNED_8(t, v) DECLARE_ALIGNED(16, t, v)
 #define STRIDE_ALIGN 16
 
 #else
@@ -662,9 +659,7 @@ extern int mm_flags;
 
 #endif
 
-#ifndef DECLARE_ALIGNED_8
 #   define DECLARE_ALIGNED_8(t, v) DECLARE_ALIGNED(8, t, v)
-#endif
 
 #ifndef STRIDE_ALIGN
 #   define STRIDE_ALIGN 8
-- 
1.6.4




More information about the ffmpeg-devel mailing list