This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] Don't compile do_test with -mavx/-mavx/-mavx512 [BZ #20384]
- From: Adhemerval Zanella <adhemerval dot zanella at linaro dot org>
- To: libc-alpha at sourceware dot org
- Date: Thu, 28 Jul 2016 09:59:28 -0300
- Subject: Re: [PATCH] Don't compile do_test with -mavx/-mavx/-mavx512 [BZ #20384]
- Authentication-results: sourceware.org; auth=none
- References: <20160719203605.GA25176@intel.com> <CAMe9rOpSeW07-ngRW9f+GyQoeOqnGV2veC61C=O2t_GyGrDi3w@mail.gmail.com>
On 27/07/2016 14:54, H.J. Lu wrote:
> On Tue, Jul 19, 2016 at 1:36 PM, H.J. Lu <hongjiu.lu@intel.com> wrote:
>> Don't compile do_test with -mavx, -mavx nor -mavx512 since they won't run
>> on non-AVX machines.
>>
>> OK for master?
>>
>> H.J.
>> ---
>> [BZ #20384]
>> * sysdeps/x86_64/fpu/Makefile (extra-test-objs): Add
>> test-double-libmvec-sincos-avx-main.o,
>> test-double-libmvec-sincos-avx2-main.o,
>> test-double-libmvec-sincos-main.o,
>> test-float-libmvec-sincosf-avx-main.o,
>> test-float-libmvec-sincosf-avx2-main.o and
>> test-float-libmvec-sincosf-main.o.
>> test-float-libmvec-sincosf-avx512-main.o.
>> ($(objpfx)test-double-libmvec-sincos): Also link with
>> $(objpfx)test-double-libmvec-sincos-main.o.
>> ($(objpfx)test-double-libmvec-sincos-avx): Also link with
>> $(objpfx)test-double-libmvec-sincos-avx-main.o.
>> ($(objpfx)test-double-libmvec-sincos-avx2): Also link with
>> $(objpfx)test-double-libmvec-sincos-avx2-main.o.
>> ($(objpfx)test-float-libmvec-sincosf): Also link with
>> $(objpfx)test-float-libmvec-sincosf-main.o.
>> ($(objpfx)test-float-libmvec-sincosf-avx): Also link with
>> $(objpfx)test-float-libmvec-sincosf-avx2-main.o.
>> [$(config-cflags-avx512) == yes] (extra-test-objs): Add
>> test-double-libmvec-sincos-avx512-main.o and
>> ($(objpfx)test-double-libmvec-sincos-avx512): Also link with
>> $(objpfx)test-double-libmvec-sincos-avx512-main.o.
>> ($(objpfx)test-float-libmvec-sincosf-avx512): Also link with
>> $(objpfx)test-float-libmvec-sincosf-avx512-main.o.
>> (CFLAGS-test-double-libmvec-sincos.c): Removed.
>> (CFLAGS-test-float-libmvec-sincosf.c): Likewise.
>> (CFLAGS-test-double-libmvec-sincos-main.c): New.
>> (CFLAGS-test-double-libmvec-sincos-avx-main.c): Likewise.
>> (CFLAGS-test-double-libmvec-sincos-avx2-main.c): Likewise.
>> (CFLAGS-test-float-libmvec-sincosf-main.c): Likewise.
>> (CFLAGS-test-float-libmvec-sincosf-avx-main.c): Likewise.
>> (CFLAGS-test-float-libmvec-sincosf-avx2-main.c): Likewise.
>> (CFLAGS-test-float-libmvec-sincosf-avx512-main.c): Likewise.
>> (CFLAGS-test-double-libmvec-sincos-avx.c): Set to -DREQUIRE_AVX.
>> (CFLAGS-test-float-libmvec-sincosf-avx.c ): Likewise.
>> (CFLAGS-test-double-libmvec-sincos-avx2.c): Set to
>> -DREQUIRE_AVX2.
>> (CFLAGS-test-float-libmvec-sincosf-avx2.c ): Likewise.
>> (CFLAGS-test-double-libmvec-sincos-avx512.c): Set to
>> -DREQUIRE_AVX512F.
>> (CFLAGS-test-float-libmvec-sincosf-avx512.c): Likewise.
>> * sysdeps/x86_64/fpu/test-double-libmvec-sincos.c: Rewritten.
>> * sysdeps/x86_64/fpu/test-float-libmvec-sincosf.c: Likewise.
>> * sysdeps/x86_64/fpu/test-double-libmvec-sincos-avx-main.c: New
>> file.
>> * sysdeps/x86_64/fpu/test-double-libmvec-sincos-avx2-main.c:
>> Likewise.
>> * sysdeps/x86_64/fpu/test-double-libmvec-sincos-avx512-main.c:
>> Likewise.
>> * sysdeps/x86_64/fpu/test-double-libmvec-sincos-main.c:
>> Likewise.
>> * sysdeps/x86_64/fpu/test-float-libmvec-sincosf-avx-main.c:
>> Likewise.
>> * sysdeps/x86_64/fpu/test-float-libmvec-sincosf-avx2-main.c:
>> Likewise.
>> * sysdeps/x86_64/fpu/test-float-libmvec-sincosf-avx512-main.c:
>> Likewise.
>> * sysdeps/x86_64/fpu/test-float-libmvec-sincosf-main.c:
>> Likewise.
>
> I am checking it in.
Ok.