gcc/ChangeLog: 2017-09-21 Sergey Shalnov <sergey.shal...@intel.com>
* config/i386/i386.c (flag_opts): Added a macro for new option (ix86_preferred_simd_mode): Prefer 256-bit AVX modes when the flag -mprefer-avx256 is on (ix86_autovectorize_vector_sizes): Avoid 512-bit vectors if the flag -mprefer-avx256 is on * config/i386/i386.opt: Doc added for -mprefer-avx256 * doc/invoke.texi: Doc added for -mprefer-avx256 gcc/testsuite/ChangeLog: 2017-09-21 Sergey Shalnov <sergey.shal...@intel.com> * g++.dg/ext/pr57362.C (__attribute__): Check for __attribute__((target("prefer-avx256"))) works * gcc.target/i386/avx512f-prefer.c: New test. Avoid 512bit registers if -mprefer-avx256 used -----Original Message----- From: Uros Bizjak [mailto:ubiz...@gmail.com] Sent: Wednesday, September 20, 2017 4:25 PM To: Shalnov, Sergey <sergey.shal...@intel.com> Cc: gcc-patches@gcc.gnu.org; kirill.yuk...@gmail.com; Koval, Julia <julia.ko...@intel.com>; Senkevich, Andrew <andrew.senkev...@intel.com> Subject: Re: [PATCH, i386] Enable option -mprefer-avx256 added for Intel AVX512 configuration On Wed, Sep 20, 2017 at 3:59 PM, Shalnov, Sergey <sergey.shal...@intel.com> wrote: > Uros, > > This is the ChangeLog > > 2017-09-20 Sergey Shalnov sergey.shal...@intel.com > * config/i386/i386.opt (mprefer-avx256): command line documentation > added for -mprefer-avx256 > * config/i386/i386.c (ix86_preferred_simd_mode): Prefer 256-bit AVX modes > when the flag -mprefer-avx256 is on. > * doc/invoke.texi Documentation about new option -mprefer-avx256 > * testsuite/g++.dg/ext/pr57362.C Added test for > __attribute__((target("prefer-avx256"))) > * testsuite/gcc.target/i386/avx512f-prefer.c New test added for the > option -mprefer-avx256 No, this is wrong style. The ChangeLog should be written like: 2017-09-20 Sergey Shalnov <sergey.shal...@intel.com> gcc/ * config/i386/i386.opt (mprefer-avx256): New option. * config/i386/i386.c (ix86_preferred_simd_mode): Return 256-bit AVX modes for TARGET_PREFER_AVX256. * doc/invoke.texi (x86 Options): Document -mprefer-avx256. testsuite/ * g++.dg/ext/pr57362.C: Test __attribute__((target("prefer-avx256"))). * gcc.target/i386/avx512f-prefer.c: New test. Please see [1] for information about general ChangeLog style and [2] about particular GCC ChangeLog requirements. [1] https://www.gnu.org/prep/standards/html_node/Change-Logs.html [2] https://gcc.gnu.org/wiki/ChangeLog Uros.