[FFmpeg-devel] [PATCH 2/3] avutil/mem_internal: define DECLARE_ALIGNED as C11's _Alignas

Zhao Zhili quinkblack at foxmail.com
Fri Nov 15 12:56:23 EET 2024


From: Zhao Zhili <zhilizhao at tencent.com>

_Alignas is portable than compiler's specific __attribute__. It do
have a limitation, that _Alignas don't support specify aligment on
the declarations of struct (it works for specify alignment on struct
fields), which only used by avcodec/cavs, and is removed now.
---
 libavutil/mem_internal.h | 26 +++++++-------------------
 1 file changed, 7 insertions(+), 19 deletions(-)

diff --git a/libavutil/mem_internal.h b/libavutil/mem_internal.h
index 20f9b3e3f2..b77157bfa0 100644
--- a/libavutil/mem_internal.h
+++ b/libavutil/mem_internal.h
@@ -73,26 +73,14 @@
  * @param v Name of the variable
  */
 
-#if defined(__INTEL_COMPILER) && __INTEL_COMPILER < 1110 || defined(__SUNPRO_C)
-    #define DECLARE_ALIGNED_T(n,t,v)    t __attribute__ ((aligned (n))) v
-    #define DECLARE_ASM_ALIGNED(n,t,v)  t __attribute__ ((aligned (n))) v
-    #define DECLARE_ASM_CONST(n,t,v)    const t __attribute__ ((aligned (n))) v
-#elif defined(__DJGPP__)
-    #define DECLARE_ALIGNED_T(n,t,v)    t __attribute__ ((aligned (FFMIN(n, 16)))) v
-    #define DECLARE_ASM_ALIGNED(n,t,v)  t av_used __attribute__ ((aligned (FFMIN(n, 16)))) v
-    #define DECLARE_ASM_CONST(n,t,v)    static const t av_used __attribute__ ((aligned (FFMIN(n, 16)))) v
-#elif defined(__GNUC__) || defined(__clang__)
-    #define DECLARE_ALIGNED_T(n,t,v)    t __attribute__ ((aligned (n))) v
-    #define DECLARE_ASM_ALIGNED(n,t,v)  t av_used __attribute__ ((aligned (n))) v
-    #define DECLARE_ASM_CONST(n,t,v)    static const t av_used __attribute__ ((aligned (n))) v
-#elif defined(_MSC_VER)
-    #define DECLARE_ALIGNED_T(n,t,v)    __declspec(align(n)) t v
-    #define DECLARE_ASM_ALIGNED(n,t,v)  __declspec(align(n)) t v
-    #define DECLARE_ASM_CONST(n,t,v)    __declspec(align(n)) static const t v
+#if defined(__DJGPP__)
+    #define DECLARE_ALIGNED_T(n,t,v)    _Alignas(FFMIN(n, 16)) t v
+    #define DECLARE_ASM_ALIGNED(n,t,v)  _Alignas(FFMIN(n, 16)) t av_used v
+    #define DECLARE_ASM_CONST(n,t,v)    _Alignas(FFMIN(n, 16)) static const t av_used v
 #else
-    #define DECLARE_ALIGNED_T(n,t,v)    t v
-    #define DECLARE_ASM_ALIGNED(n,t,v)  t v
-    #define DECLARE_ASM_CONST(n,t,v)    static const t v
+    #define DECLARE_ALIGNED_T(n,t,v)    _Alignas(n) t v
+    #define DECLARE_ASM_ALIGNED(n,t,v)  _Alignas(n) t av_used v
+    #define DECLARE_ASM_CONST(n,t,v)    _Alignas(n) static const t av_used v
 #endif
 
 #if HAVE_SIMD_ALIGN_64
-- 
2.25.1



More information about the ffmpeg-devel mailing list