[FFmpeg-cvslog] h264/10bit: add HAVE_ALIGNED_STACK checks.

Daniel Kang git at videolan.org
Thu Jun 2 05:18:57 CEST 2011


ffmpeg | branch: master | Daniel Kang <daniel.d.kang at gmail.com> | Tue May 31 23:10:51 2011 -0400| [f3aa65af3a84c787b0fa8b8cf0881fc05668a24c] | committer: Ronald S. Bultje

h264/10bit: add HAVE_ALIGNED_STACK checks.

Fixes regression in 836f47d34b49e8ba9883e738a42f154130421caa in ICC-10.x,
since ICC<=11.0 doesn't align stack upon function calls.

Signed-off-by: Ronald S. Bultje <rsbultje at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=f3aa65af3a84c787b0fa8b8cf0881fc05668a24c
---

 libavcodec/x86/h264dsp_mmx.c |   12 ++++++++----
 1 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/libavcodec/x86/h264dsp_mmx.c b/libavcodec/x86/h264dsp_mmx.c
index 1a31e41..3fccd08 100644
--- a/libavcodec/x86/h264dsp_mmx.c
+++ b/libavcodec/x86/h264dsp_mmx.c
@@ -445,12 +445,14 @@ void ff_h264dsp_init_x86(H264DSPContext *c, const int bit_depth)
             if (mm_flags&AV_CPU_FLAG_SSE2) {
                 c->h264_idct_add       = ff_h264_idct_add_10_sse2;
                 c->h264_idct8_dc_add   = ff_h264_idct8_dc_add_10_sse2;
-                c->h264_idct8_add      = ff_h264_idct8_add_10_sse2;
 
                 c->h264_idct_add16     = ff_h264_idct_add16_10_sse2;
-                c->h264_idct8_add4     = ff_h264_idct8_add4_10_sse2;
                 c->h264_idct_add8      = ff_h264_idct_add8_10_sse2;
                 c->h264_idct_add16intra= ff_h264_idct_add16intra_10_sse2;
+#if HAVE_ALIGNED_STACK
+                c->h264_idct8_add      = ff_h264_idct8_add_10_sse2;
+                c->h264_idct8_add4     = ff_h264_idct8_add4_10_sse2;
+#endif
 
                 c->h264_v_loop_filter_chroma= ff_deblock_v_chroma_10_sse2;
                 c->h264_v_loop_filter_chroma_intra= ff_deblock_v_chroma_intra_10_sse2;
@@ -465,13 +467,15 @@ void ff_h264dsp_init_x86(H264DSPContext *c, const int bit_depth)
             if (mm_flags&AV_CPU_FLAG_AVX) {
                 c->h264_idct_dc_add    =
                 c->h264_idct_add       = ff_h264_idct_add_10_avx;
-                c->h264_idct8_add      = ff_h264_idct8_add_10_avx;
                 c->h264_idct8_dc_add   = ff_h264_idct8_dc_add_10_avx;
 
                 c->h264_idct_add16     = ff_h264_idct_add16_10_avx;
-                c->h264_idct8_add4     = ff_h264_idct8_add4_10_avx;
                 c->h264_idct_add8      = ff_h264_idct_add8_10_avx;
                 c->h264_idct_add16intra= ff_h264_idct_add16intra_10_avx;
+#if HAVE_ALIGNED_STACK
+                c->h264_idct8_add      = ff_h264_idct8_add_10_avx;
+                c->h264_idct8_add4     = ff_h264_idct8_add4_10_avx;
+#endif
 
                 c->h264_v_loop_filter_chroma= ff_deblock_v_chroma_10_avx;
                 c->h264_v_loop_filter_chroma_intra= ff_deblock_v_chroma_intra_10_avx;



More information about the ffmpeg-cvslog mailing list