[GCC][Patch]Bug fix: cannot convert 'const short int*' to 'const __bf16*'

Kyrill Tkachov kyrylo.tkachov@foss.arm.com
Wed Mar 11 18:03:00 GMT 2020


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.burduv@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
>
>
>


More information about the Gcc-patches mailing list