[PATCH v2 15/28] x86: Use -mavx instead of -msse2avx

[Adhemerval Zanella Netto]

On 28/10/25 16:02, H.J. Lu wrote:
> On Wed, Oct 29, 2025 at 1:13 AM Adhemerval Zanella
> <adhemerval.zanella@linaro.org> wrote:
>>
>> clang supports it from version 19 and onwards, but it should be gated
>> as an option to assembler (either with -Wa or -Xassembler).  Add a
>> test for -Wa as well if -msse2avx fully supported by the compiler.
>> ---
>>  sysdeps/x86_64/fpu/multiarch/Makefile | 14 +++++++-------
>>  1 file changed, 7 insertions(+), 7 deletions(-)
>>
>> diff --git a/sysdeps/x86_64/fpu/multiarch/Makefile b/sysdeps/x86_64/fpu/multiarch/Makefile
>> index 708b14297f..bd887cd59c 100644
>> --- a/sysdeps/x86_64/fpu/multiarch/Makefile
>> +++ b/sysdeps/x86_64/fpu/multiarch/Makefile
>> @@ -150,13 +150,13 @@ CFLAGS-s_sin-fma4.c = -mfma4
>>  CFLAGS-s_tan-fma4.c = -mfma4
>>  CFLAGS-s_sincos-fma4.c = -mfma4
>>
>> -CFLAGS-e_atan2-avx.c = -msse2avx -DSSE2AVX
>> -CFLAGS-e_exp-avx.c = -msse2avx -DSSE2AVX
>> -CFLAGS-e_log-avx.c = -msse2avx -DSSE2AVX
>> -CFLAGS-s_atan-avx.c = -msse2avx -DSSE2AVX
>> -CFLAGS-s_sin-avx.c = -msse2avx -DSSE2AVX
>> -CFLAGS-s_tan-avx.c = -msse2avx -DSSE2AVX
>> -CFLAGS-s_sincos-avx.c = -msse2avx -DSSE2AVX
>> +CFLAGS-e_atan2-avx.c = -mavx -DSSE2AVX
>> +CFLAGS-e_exp-avx.c = -mavx -DSSE2AVX
>> +CFLAGS-e_log-avx.c = -mavx -DSSE2AVX
>> +CFLAGS-s_atan-avx.c = -mavx -DSSE2AVX
>> +CFLAGS-s_sin-avx.c = -mavx -DSSE2AVX
>> +CFLAGS-s_tan-avx.c = -mavx -DSSE2AVX
>> +CFLAGS-s_sincos-avx.c = -mavx -DSSE2AVX
> 
> -DSSE2AVX shouldn't be necessary.

I ended up re-using this on clang optimization for AVX [1].

[1] https://patchwork.sourceware.org/project/glibc/patch/20251028171141.3208834-19-adhemerval.zanella@linaro.org/

> 
>>
>>  CFLAGS-s_modf-sse4_1.c = -msse4.1 -fno-builtin-modff32x -fno-builtin-modff64
>>  CFLAGS-s_modff-sse4_1.c = -msse4.1 -fno-builtin-modff32
>> --
>> 2.43.0
>>
> 
>