On Wed, Jan 09, 2013 at 10:57:41PM +0200, Martin Storsjö wrote:
> This is a fallback for cases where building with yasm enabled, but
> the compiler doesn't support inline asm (or it is disabled by the
> user), and the compiler doesn't support the _mm_empty intrinsic (like
> gcc, if the user hasn't added -mmmx to the cflags).

I stumbled a bit while reading this paragraph, here's my try:

  x86: Add a yasm-based emms() replacement

  This provides a fallback when building with Yasm enabled, but neither
  inline assembly, nor the _mm_empty intrinsic are available or enabled.

> The function is named emms_c - if it were named plain 'emms' it
> causes conflicts with the instruction with the same name within
> the function.

emms_yasm or emms_asm might be a bit more descriptive.

> --- /dev/null
> +++ b/libavutil/x86/emms.asm
> @@ -0,0 +1,34 @@
> +
> +%define program_name avpriv
> +
> +%include "x86util.asm"

Maybe include x86inc.asm, then you could skip 1/3.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to