Commit d82d1139 authored by Michael Niedermayer's avatar Michael Niedermayer

avutil/arm/intmath: return int for uint8 / uint16 clip

The C functions return uint8/16_t but that is effectively int not unsigned int
Fixes fate-filter-tblend

We do not return uint8/16_t as that would require the compiler to truncate the
values, slowing it down.
Signed-off-by: 's avatarMichael Niedermayer <michael@niedermayer.cc>
parent f85bc147
...@@ -31,9 +31,9 @@ ...@@ -31,9 +31,9 @@
#if HAVE_ARMV6_INLINE #if HAVE_ARMV6_INLINE
#define av_clip_uint8 av_clip_uint8_arm #define av_clip_uint8 av_clip_uint8_arm
static av_always_inline av_const unsigned av_clip_uint8_arm(int a) static av_always_inline av_const int av_clip_uint8_arm(int a)
{ {
unsigned x; int x;
__asm__ ("usat %0, #8, %1" : "=r"(x) : "r"(a)); __asm__ ("usat %0, #8, %1" : "=r"(x) : "r"(a));
return x; return x;
} }
...@@ -47,9 +47,9 @@ static av_always_inline av_const int av_clip_int8_arm(int a) ...@@ -47,9 +47,9 @@ static av_always_inline av_const int av_clip_int8_arm(int a)
} }
#define av_clip_uint16 av_clip_uint16_arm #define av_clip_uint16 av_clip_uint16_arm
static av_always_inline av_const unsigned av_clip_uint16_arm(int a) static av_always_inline av_const int av_clip_uint16_arm(int a)
{ {
unsigned x; int x;
__asm__ ("usat %0, #16, %1" : "=r"(x) : "r"(a)); __asm__ ("usat %0, #16, %1" : "=r"(x) : "r"(a));
return x; return x;
} }
......
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