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
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
