Commit 1bdb47b7 authored by Daniel Kolesa's avatar Daniel Kolesa Committed by Lauri Kasanen

swscale: Replace illegal vector keyword usage in altivec code

While this technically compiles in current ffmpeg, this is only
because ffmpeg is compiled in strict ISO C mode, which disables
the builtin 'vector' keyword for AltiVec/VSX. Instead this gets
replaced with a macro inside altivec.h, which defines vector to
be actually __vector, which accepts random types.

Normally, the vector keyword should be used only with plain
scalar non-typedef types, such as unsigned int. But we have the
vec_(s|u)(8|16|32) macros, which can be used in a portable manner,
in util_altivec.h in libavutil.

This is also consistent with other AltiVec/VSX code elsewhere in
the tree.

Fixes #7861.
Signed-off-by: 's avatarDaniel Kolesa <daniel@octaforge.org>
Signed-off-by: 's avatarLauri Kasanen <cand@gmx.com>
parent 581419ea
...@@ -153,13 +153,13 @@ static void yuv2plane1_float_altivec(const int32_t *src, float *dest, int dstW) ...@@ -153,13 +153,13 @@ static void yuv2plane1_float_altivec(const int32_t *src, float *dest, int dstW)
const int add = (1 << (shift - 1)); const int add = (1 << (shift - 1));
const int clip = (1 << 16) - 1; const int clip = (1 << 16) - 1;
const float fmult = 1.0f / 65535.0f; const float fmult = 1.0f / 65535.0f;
const vector uint32_t vadd = (vector uint32_t) {add, add, add, add}; const vec_u32 vadd = (vec_u32) {add, add, add, add};
const vector uint32_t vshift = (vector uint32_t) vec_splat_u32(shift); const vec_u32 vshift = (vec_u32) vec_splat_u32(shift);
const vector uint32_t vlargest = (vector uint32_t) {clip, clip, clip, clip}; const vec_u32 vlargest = (vec_u32) {clip, clip, clip, clip};
const vector float vmul = (vector float) {fmult, fmult, fmult, fmult}; const vec_f vmul = (vec_f) {fmult, fmult, fmult, fmult};
const vector float vzero = (vector float) {0, 0, 0, 0}; const vec_f vzero = (vec_f) {0, 0, 0, 0};
vector uint32_t v; vec_u32 v;
vector float vd; vec_f vd;
int i; int i;
yuv2plane1_float_u(src, dest, dst_u, 0); yuv2plane1_float_u(src, dest, dst_u, 0);
...@@ -186,15 +186,15 @@ static void yuv2plane1_float_bswap_altivec(const int32_t *src, uint32_t *dest, i ...@@ -186,15 +186,15 @@ static void yuv2plane1_float_bswap_altivec(const int32_t *src, uint32_t *dest, i
const int add = (1 << (shift - 1)); const int add = (1 << (shift - 1));
const int clip = (1 << 16) - 1; const int clip = (1 << 16) - 1;
const float fmult = 1.0f / 65535.0f; const float fmult = 1.0f / 65535.0f;
const vector uint32_t vadd = (vector uint32_t) {add, add, add, add}; const vec_u32 vadd = (vec_u32) {add, add, add, add};
const vector uint32_t vshift = (vector uint32_t) vec_splat_u32(shift); const vec_u32 vshift = (vec_u32) vec_splat_u32(shift);
const vector uint32_t vlargest = (vector uint32_t) {clip, clip, clip, clip}; const vec_u32 vlargest = (vec_u32) {clip, clip, clip, clip};
const vector float vmul = (vector float) {fmult, fmult, fmult, fmult}; const vec_f vmul = (vec_f) {fmult, fmult, fmult, fmult};
const vector float vzero = (vector float) {0, 0, 0, 0}; const vec_f vzero = (vec_f) {0, 0, 0, 0};
const vector uint32_t vswapbig = (vector uint32_t) {16, 16, 16, 16}; const vec_u32 vswapbig = (vec_u32) {16, 16, 16, 16};
const vector uint16_t vswapsmall = vec_splat_u16(8); const vec_u16 vswapsmall = vec_splat_u16(8);
vector uint32_t v; vec_u32 v;
vector float vd; vec_f vd;
int i; int i;
yuv2plane1_float_bswap_u(src, dest, dst_u, 0); yuv2plane1_float_bswap_u(src, dest, dst_u, 0);
...@@ -208,8 +208,8 @@ static void yuv2plane1_float_bswap_altivec(const int32_t *src, uint32_t *dest, i ...@@ -208,8 +208,8 @@ static void yuv2plane1_float_bswap_altivec(const int32_t *src, uint32_t *dest, i
vd = vec_ctf(v, 0); vd = vec_ctf(v, 0);
vd = vec_madd(vd, vmul, vzero); vd = vec_madd(vd, vmul, vzero);
vd = (vector float) vec_rl((vector uint32_t) vd, vswapbig); vd = (vec_f) vec_rl((vec_u32) vd, vswapbig);
vd = (vector float) vec_rl((vector uint16_t) vd, vswapsmall); vd = (vec_f) vec_rl((vec_u16) vd, vswapsmall);
vec_st(vd, 0, (float *) &dest[i]); vec_st(vd, 0, (float *) &dest[i]);
} }
......
This diff is collapsed.
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment