On Fri, 24 Jul 2020 10:59:35 +0200
Felix Willgerodt via Gdb-patches <gdb-patc...@sourceware.org> wrote:

>       * features/i386/64bit-avx512.c
>       (create_feature_i386_64bit_avx512): Add vector v8bf.
>       (create_feature_i386_64bit_avx512): Add field v8_bfloat16.
>       * features/i386/64bit-avx512.xml: Likewise.
>       * features/i386/64bit-sse.c
>       (create_feature_i386_64bit_sse): Add vector v8bf.
>       (create_feature_i386_64bit_sse): Add field v8_bfloat16.
>       * features/i386/64bit-sse.xml: Likewise.

For these ChangeLog entries, I'd prefer to see:

        * features/i386/64bit-avx512.xml
        (create_feature_i386_64bit_avx512): Add vector v8bf.
        (create_feature_i386_64bit_avx512): Add field v8_bfloat16.
        * features/i386/64bit-avx512.c: Regenerate.
        * features/i386/64bit-sse.xml
        (create_feature_i386_64bit_sse): Add vector v8bf.
        (create_feature_i386_64bit_sse): Add field v8_bfloat16.
        * features/i386/64bit-sse.c: Regenerate.

Otherwise, okay.

(Except for the libiberty and include portions which need approval
from a GCC maintainer.)

Kevin

Reply via email to