This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, i386] Add new arg values for __builtin_cpu_supports
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Cc: Ilya Tocar <tocarip dot intel at gmail dot com>, Jakub Jelinek <jakub at redhat dot com>, Jeff Law <law at redhat dot com>, ÐÐÑÑ ÐÐÐÐÐÐÑ <enkovich dot gnu at gmail dot com>
- Date: Wed, 26 Nov 2014 19:46:26 +0100
- Subject: Re: [PATCH, i386] Add new arg values for __builtin_cpu_supports
- Authentication-results: sourceware.org; auth=none
Hello!
> I've added avx512f support to __builtin_cpu_supports.
> I'm not sure about bw+vl, i think for compound values like
> avx512bd+dq+vl, arch is better. Also for such cases prority is unclear,
> what should we choose bw+vl or e. g. avx512f+er?
> I've left MPX bits in cpuid.h, in case we will need them later (e. g.
> for runtime mpx tests enabling).
>
> Ok for trunk?
>
> gcc/
>
> * config/i386/cpuid.h (bit_MPX, bit_BNDREGS, bit_BNDCSR):
> Define.
> * config/i386/i386.c (get_builtin_code_for_version): Add avx512f.
> (fold_builtin_cpu): Ditto.
> * doc/extend.texi: Documment it.
>
>
> gcc/testsuite/
>
> * g++.dg/ext/mv2.C: Add test for target ("avx512f").
> * gcc.target/i386/builtin_target.c: Ditto.
>
>
> libgcc/
>
> * config/i386/cpuinfo.c (processor_features): Add FEATURE_AVX512F.
> * config/i386/cpuinfo.c (get_available_features): Detect it.
OK.
Thanks,
Uros.