On 14/04/2018 19:38, Martin Storsjö wrote:
> Make the actual implementation static inline, but add a non-static
> non-inline frontend for it.
> 
> This fixes building with clang in msvc mode, which does support
> gcc style inline assembly.
> ---
>  libavcodec/x86/xvididct_sse2.c | 11 ++++++++---
>  1 file changed, 8 insertions(+), 3 deletions(-)
> 
> diff --git a/libavcodec/x86/xvididct_sse2.c b/libavcodec/x86/xvididct_sse2.c
> index f318e95..00ed803 100644
> --- a/libavcodec/x86/xvididct_sse2.c
> +++ b/libavcodec/x86/xvididct_sse2.c
> @@ -342,7 +342,7 @@ DECLARE_ASM_CONST(16, int32_t, walkenIdctRounders)[] = {
>      "movdqa   %%xmm6, 4*16("dct")     \n\t" \
>      "movdqa   "SREG2", 7*16("dct")    \n\t"
>  
> -inline void ff_xvid_idct_sse2(short *block)
> +static inline void xvid_idct_sse2(short *block)
>  {
>      __asm__ volatile (
>          "movq     "MANGLE (m127) ", %%mm0                              \n\t"
> @@ -390,15 +390,20 @@ inline void ff_xvid_idct_sse2(short *block)
>            "%eax", "%ecx", "%edx", "%esi", "memory");
>  }
>  
> +void ff_xvid_idct_sse2(short *block)
> +{
> +    xvid_idct_sse2(block);
> +}
> +
>  void ff_xvid_idct_sse2_put(uint8_t *dest, ptrdiff_t line_size, short *block)
>  {
> -    ff_xvid_idct_sse2(block);
> +    xvid_idct_sse2(block);
>      ff_put_pixels_clamped_mmx(block, dest, line_size);
>  }
>  
>  void ff_xvid_idct_sse2_add(uint8_t *dest, ptrdiff_t line_size, short *block)
>  {
> -    ff_xvid_idct_sse2(block);
> +    xvid_idct_sse2(block);
>      ff_add_pixels_clamped_mmx(block, dest, line_size);
>  }
>  
> 

Sure
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to