This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH i386 AVX512] [85/n] Add intrinsics headers.
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: Kirill Yukhin <kirill dot yukhin at gmail dot com>
- Cc: Jakub Jelinek <jakub at redhat dot com>, Richard Henderson <rth at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 23 Oct 2014 20:13:57 +0200
- Subject: Re: [PATCH i386 AVX512] [85/n] Add intrinsics headers.
- Authentication-results: sourceware.org; auth=none
- References: <20141022124319 dot GH11644 at msticlxl57 dot ims dot intel dot com>
On Wed, Oct 22, 2014 at 2:43 PM, Kirill Yukhin <kirill.yukhin@gmail.com> wrote:
> Hello,
> This patch introduces necessary AVX-512* intrinsics headers.
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/avx512bwintrin.h: New.
> * config/i386/avx512dqintrin.h: Ditto.
> * config/i386/avx512vlbwintrin.h: Ditto.
> * config/i386/avx512vldqintrin.h: Ditto.
> * config/i386/avx512vlintrin.h: Ditto.
> * config/i386/immintrin.h: Include avx512vlintrin.h, avx512bwintrin.h,
> avx512dqintrin.h, avx512vlbwintrin.h, avx512vldqintrin.h.
Well, I have checked that patch follows the established coding style,
that it has #defines that cover constant propagation for -O0 and has
(hopefully) correct target flags, and that is the maximum a human can
do. Testsuite (mostly gcc.target/i386/sse-{12,13, 14}.c and
g++.dg/other/i386-{2,3}.C will cover the rest.
Oh, a couple of headers still have 2013 for their copyright data.
Please change these to 2014.
Rubber stamped as OK.
Thanks,
Uros.