lame-3.100: Tweak for i?68 does not work

[thomas]

There is the instruction

case $(uname -m) in
   i?86) sed -i -e '/xmmintrin\.h/d' configure ;;
esac

meant to prevent from using the xmmintrin.h file which results in error on i?86 systems.

It does even with this sed:

In file included from xmm_quantize_sub.c:37:0:
/usr/lib/gcc/i686-pc-linux-gnu/7.3.0/include/xmmintrin.h:240:1: Fehler: »inline« beim Aufruf von always_inline »_mm_and_ps« gescheitert: target specific option mismatch
_mm_and_ps (__m128 __A, __m128 __B)
 ^~~~~~~~~~
xmm_quantize_sub.c:96:29: Anmerkung: von hier aufgerufen
             vec_tmp._m128 = _mm_and_ps(vec_tmp._m128, vec_fabs_mask); /* fabs */
                             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
make[3]: *** [Makefile:423: xmm_quantize_sub.lo] Fehler 1
make[3]: Verzeichnis „/tmp/lame/build/lame-3.100/libmp3lame/vector“ wird verlassen
make[2]: *** [Makefile:613: all-recursive] Fehler 1
make[2]: Verzeichnis „/tmp/lame/build/lame-3.100/libmp3lame“ wird verlassen
make[1]: *** [Makefile:452: all-recursive] Fehler 1
make[1]: Verzeichnis „/tmp/lame/build/lame-3.100“ wird verlassen
make: *** [Makefile:381: all] Fehler 2

No issue on x86_64.

[thomas]

looks like that

case $(uname -m) in
    i?86) sed -i -e 's/<xmmintrin.h/&.nouse/' configure ;;
esac

brings the expected result.

Any objections?

[bdubbs@…]

Just a guess, but try removing the line:

#define HAVE_XMMINTRIN_H 1

after configure instead of the sed.

[thomas]

just commited the fix above, will try yours, too.

[thomas]

Yes, doing a sed -e "/HAVE_XMMINTRIN_H/d" -i config.h after configure has run does the trick, too.