<p dir="ltr">Correct. I'll follow up with a correct patch tomorrow. </p>
<br><div class="gmail_quote"><div dir="ltr">On Sat, 5 Dec 2015, 19:03 Rémi Denis-Courmont <<a href="mailto:remi@remlab.net">remi@remlab.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le 2015-12-05 18:38, Diego Elio Petten� a écrit :<br>
> From: Diego Elio Pettenà<<a href="mailto:flameeyes@flameeyes.eu" target="_blank">flameeyes@flameeyes.eu</a>><br>
><br>
> This disables MMX/MMXEXT code for ffmpeg/postproc, and all assembly<br>
> for mpg123 and libass.<br>
><br>
> It has performance implications but it avoids all TEXTRELs in the<br>
> output, which makes it possible to use W^X techniques. The code can<br>
> probably be ported to be PIC-compatible in the future so that these<br>
> can be re-enabled, but even Gentoo Linux has no better solutions for<br>
> this at the moment.<br>
> ---<br>
>  contrib/src/ass/rules.mak      | 5 +++++<br>
>  contrib/src/ffmpeg/rules.mak   | 4 ++++<br>
>  contrib/src/mpg123/rules.mak   | 4 ++++<br>
>  contrib/src/postproc/rules.mak | 4 ++++<br>
>  4 files changed, 17 insertions(+)<br>
><br>
> diff --git a/contrib/src/ass/rules.mak b/contrib/src/ass/rules.mak<br>
> index a9c2603..d2337aa 100644<br>
> --- a/contrib/src/ass/rules.mak<br>
> +++ b/contrib/src/ass/rules.mak<br>
> @@ -10,6 +10,7 @@ endif<br>
>  ifdef HAVE_ANDROID<br>
>  WITH_FONTCONFIG = 0<br>
>  WITH_HARFBUZZ = 0<br>
> +WITH_ASM = 0<br>
<br>
Pre-existing problem but this clobbers the Makefile namespace.<br>
<br>
>  else<br>
>  ifdef HAVE_TIZEN<br>
>  WITH_FONTCONFIG = 0<br>
> @@ -59,6 +60,10 @@ else<br>
>  ASS_CONF += --disable-harfbuzz<br>
>  endif<br>
><br>
> +ifeq ($(WITH_ASM), 0)<br>
> +ASS_CONF += --disable-asm<br>
> +endif<br>
> +<br>
>  ifdef WITH_OPTIMIZATION<br>
>  ASS_CFLAGS += -O3<br>
>  else<br>
> diff --git a/contrib/src/ffmpeg/rules.mak<br>
> b/contrib/src/ffmpeg/rules.mak<br>
> index 032d66a..3bdf1ad 100644<br>
> --- a/contrib/src/ffmpeg/rules.mak<br>
> +++ b/contrib/src/ffmpeg/rules.mak<br>
> @@ -138,6 +138,10 @@ FFMPEGCONF += --target-os=linux --enable-pic<br>
><br>
>  endif<br>
><br>
> +ifdef HAVE_ANDROID<br>
> +FFMPEGCONF +=  --disable-mmx --disable-mmxext<br>
> +endif<br>
> +<br>
>  # Windows<br>
>  ifdef HAVE_WIN32<br>
>  ifndef HAVE_MINGW_W64<br>
> diff --git a/contrib/src/mpg123/rules.mak<br>
> b/contrib/src/mpg123/rules.mak<br>
> index 1f90f79..66a82a3 100644<br>
> --- a/contrib/src/mpg123/rules.mak<br>
> +++ b/contrib/src/mpg123/rules.mak<br>
> @@ -7,6 +7,10 @@ ifeq ($(call need_pkg,"mpg123"),)<br>
>  PKGS_FOUND += mpg123<br>
>  endif<br>
><br>
> +ifdef HAVE_ANDROID<br>
> +HOSTCONF += --with-cpu=generic_fpu<br>
> +endif<br>
> +<br>
>  $(TARBALLS)/mpg123-$(MPG123_VERSION).tar.bz2:<br>
>       $(call download,$(MPG123_URL))<br>
><br>
> diff --git a/contrib/src/postproc/rules.mak<br>
> b/contrib/src/postproc/rules.mak<br>
> index a132e57..90cdf86 100644<br>
> --- a/contrib/src/postproc/rules.mak<br>
> +++ b/contrib/src/postproc/rules.mak<br>
> @@ -73,6 +73,10 @@ ifdef HAVE_LINUX<br>
>  POSTPROCCONF += --target-os=linux --enable-pic<br>
>  endif<br>
><br>
> +ifdef HAVE_ANDROID<br>
> +POSTPROCCONF +=  --disable-mmx --disable-mmxext<br>
> +endif<br>
> +<br>
>  # Windows<br>
>  ifdef HAVE_WIN32<br>
>  POSTPROCCONF += --target-os=mingw32<br>
<br>
--<br>
Rémi Denis-Courmont<br>
<a href="http://www.remlab.net/" rel="noreferrer" target="_blank">http://www.remlab.net/</a><br>
</blockquote></div><div dir="ltr">-- <br></div><div dir="ltr">Diego Elio Pettenò (aka Flameeyes)</div>