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.

Reply via email to