[FFmpeg-devel] Fix bug: libavcodec/ppc/h264dsp.c

[Michael Niedermayer]

On Thu, May 14, 2015 at 03:31:15PM +0800, rongyan wrote:
> Hi,
>  We got defective video when use GCC 4.9.2 instead of GCC 4.9.1 to compile FFMEPG. And further found that GCC 4.8 and 4.9 need patch to fix the lvsl/lvsr bug on POWER LE, and GCC 5.1 contains the correct code since its release. The message on gcc-patches requesting approval for lvsl/lvsr patch is at https://gcc.gnu.org/ml/gcc-patches/2014-10/msg00228.html.
>   
>  So we create this patch to fix the lvsl bug for libavcodec/ppc/h264dsp.c, the fixed code avoid using lvsl and will not depends on GCC version, also it uses less instructions on POWER LE.
>   
>  The fate test before fixing is:
>  
>   
>  And after fixing is:
>  
>   
>  Thanks for your review.
>   
>  Rong Yan
>   
>  ------------------  
>    The world has enough for everyone's need, but not enough for everyone's greed.



>  h264dsp.c |   26 +++++++++++++++-----------
>  1 file changed, 15 insertions(+), 11 deletions(-)
> 52aeb0bb08d955232f344bb4192365b1a359f9d8  0001-avcodec-ppc-h264dsp-POWER-LE-support-in-h264_idct_dc.patch
> From e0a432223f1b7b26afd15e2c08faebe6dc8e15f8 Mon Sep 17 00:00:00 2001
> From: Rong Yan <rongyan236 at gmail.com>
> Date: Thu, 14 May 2015 06:43:44 +0000
> Subject: [PATCH] avcodec/ppc/h264dsp: POWER LE support in
>  h264_idct_dc_add_internal() fix vec_lvsl bug

applied

thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are best at talking, realize last or never when they are wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150514/189a9925/attachment.asc>