On 3/11/20 5:59 PM, Kyrill Tkachov wrote:
Hi Delia,

On 3/11/20 5:49 PM, Delia Burduv wrote:
This patch fixes a bug introduced by my earlier patch (
https://gcc.gnu.org/pipermail/gcc-patches/2020-March/541680.html ).
It introduces a new scalar builtin type that was missing in the original
patch.

Bootstrapped cleanly on arm-none-linux-gnueabihf.
Tested for regression on arm-none-linux-gnueabihf. No regression from
before the original patch.
Tests that failed or became unsupported because of the original tests
now work as they did before it.

gcc/ChangeLog:

2020-03-11  Delia Burduv  <delia.bur...@arm.com>

        * config/arm/arm-builtins.c
         (arm_init_simd_builtin_scalar_types): New
        * config/arm/arm_neon.h (vld2_bf16): Used new builtin type
        (vld2q_bf16): Used new builtin type
        (vld3_bf16): Used new builtin type
        (vld3q_bf16): Used new builtin type
        (vld4_bf16): Used new builtin type
        (vld4q_bf16): Used new builtin type
        (vld2_dup_bf16): Used new builtin type
        (vld2q_dup_bf16): Used new builtin type
        (vld3_dup_bf16): Used new builtin type
        (vld3q_dup_bf16): Used new builtin type
        (vld4_dup_bf16): Used new builtin type
        (vld4q_dup_bf16): Used new builtin type

ChangeLog entries should have a full stop after each entry.

The patch is ok.

Thanks for the quick fix,


To be clear, I've pushed it to master with a fixed ChangeLog as 1c43ee69f4f6148fff4b5ace80d709d7f8b250d7

Kyrill



Kyrill



Reply via email to