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.