aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMans Rullgard <mans@mansr.com>2011-02-21 19:15:38 +0000
committerMichael Niedermayer <michaelni@gmx.at>2011-02-22 02:44:39 +0100
commite407f4173a0ab89fe9ab57e228f7ec0467a24ca4 (patch)
tree691edfa92cabbbca8a3c72f7ca33d45438db36a4
parent2df9d0008ee34f99212f8c9820bd858ff8b64642 (diff)
downloadffmpeg-e407f4173a0ab89fe9ab57e228f7ec0467a24ca4.tar.gz
ppc: fix vc1 inverse transform, unbreak build
GCC 4.3 and later are more particular about signedness matching in vector operations. The operations under if(rangered) were missing assignments and thus had no effect. Signed-off-by: Mans Rullgard <mans@mansr.com> (cherry picked from commit 381efba0ecedd41575f99ba9e9bd3826551079f6)
-rw-r--r--libavcodec/ppc/vc1dsp_altivec.c34
1 files changed, 17 insertions, 17 deletions
diff --git a/libavcodec/ppc/vc1dsp_altivec.c b/libavcodec/ppc/vc1dsp_altivec.c
index b5cc71c3cf..05edb53b7c 100644
--- a/libavcodec/ppc/vc1dsp_altivec.c
+++ b/libavcodec/ppc/vc1dsp_altivec.c
@@ -146,7 +146,7 @@ static void vc1_inv_trans_8x8_altivec(DCTELEM block[64],
const vector signed int vec_1s = vec_splat_s32(1);
const vector unsigned int vec_1 = vec_splat_u32(1);
const vector unsigned short rangered_shift = vec_splat_u16(1);
- const vector signed short signed_bias = vec_sl(vec_splat_u16(4),
+ const vector signed short signed_bias = vec_sl(vec_splat_s16(4),
vec_splat_u16(4));
src0 = vec_ld( 0, block);
@@ -219,23 +219,23 @@ static void vc1_inv_trans_8x8_altivec(DCTELEM block[64],
if (rangered) {
if (!sign) {
- vec_sub(src0, signed_bias);
- vec_sub(src1, signed_bias);
- vec_sub(src2, signed_bias);
- vec_sub(src3, signed_bias);
- vec_sub(src4, signed_bias);
- vec_sub(src5, signed_bias);
- vec_sub(src6, signed_bias);
- vec_sub(src7, signed_bias);
+ src0 = vec_sub(src0, signed_bias);
+ src1 = vec_sub(src1, signed_bias);
+ src2 = vec_sub(src2, signed_bias);
+ src3 = vec_sub(src3, signed_bias);
+ src4 = vec_sub(src4, signed_bias);
+ src5 = vec_sub(src5, signed_bias);
+ src6 = vec_sub(src6, signed_bias);
+ src7 = vec_sub(src7, signed_bias);
}
- vec_sl(src0, rangered_shift);
- vec_sl(src1, rangered_shift);
- vec_sl(src2, rangered_shift);
- vec_sl(src3, rangered_shift);
- vec_sl(src4, rangered_shift);
- vec_sl(src5, rangered_shift);
- vec_sl(src6, rangered_shift);
- vec_sl(src7, rangered_shift);
+ src0 = vec_sl(src0, rangered_shift);
+ src1 = vec_sl(src1, rangered_shift);
+ src2 = vec_sl(src2, rangered_shift);
+ src3 = vec_sl(src3, rangered_shift);
+ src4 = vec_sl(src4, rangered_shift);
+ src5 = vec_sl(src5, rangered_shift);
+ src6 = vec_sl(src6, rangered_shift);
+ src7 = vec_sl(src7, rangered_shift);
}
vec_st(src0, 0, block);