On Wed, Oct 22, 2014 at 2:43 PM, Kirill Yukhin <kirill.yuk...@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.