Christophe Lyon <christophe.l...@linaro.org> writes:

> On Wed, 4 Nov 2020 at 14:29, Christophe Lyon <christophe.l...@linaro.org> 
> wrote:
>>
>> On Tue, 3 Nov 2020 at 11:27, Kyrylo Tkachov via Gcc-patches
>> <gcc-patches@gcc.gnu.org> wrote:
>> >
>> > Hi Andrea,
>> >
>> > > -----Original Message-----
>> > > From: Andrea Corallo <andrea.cora...@arm.com>
>> > > Sent: 26 October 2020 15:59
>> > > To: gcc-patches@gcc.gnu.org
>> > > Cc: Kyrylo Tkachov <kyrylo.tkac...@arm.com>; Richard Earnshaw
>> > > <richard.earns...@arm.com>; nd <n...@arm.com>
>> > > Subject: [PATCH 1/x] arm: Add vld1_lane_bf16 + vldq_lane_bf16 intrinsics
>> > >
>> > > Hi all,
>> > >
>> > > I'd like to submit the following patch implementing the bfloat16_t
>> > > neon related load intrinsics: vld1_lane_bf16, vld1q_lane_bf16.
>> > >
>> > > Please see refer to:
>> > > ACLE <https://developer.arm.com/docs/101028/latest>
>> > > ISA  <https://developer.arm.com/docs/ddi0596/latest>
>> > >
>> > > Regtested and bootstrapped.
>> > >
>> > > Okay for trunk?
>> >
>>
>> I think you need to add -mfloat-abi=hard to the dg-additional-options
>> otherwise vld1_lane_bf16_1.c
>> fails on targets with a soft float-abi default (eg arm-linux-gnueabi).
>>
>> See bf16_vldn_1.c.
>
> Actually that's not sufficient because in turn we get:
> /sysroot-arm-none-linux-gnueabi/usr/include/gnu/stubs.h:10:11: fatal
> error: gnu/stubs-hard.h: No such file or directory
>
> So you should check that -mfloat-abi=hard is supported.
>
> Ditto for the vst tests.
>

Hi Christophe,

thanks for catching this, I'll prepare a patch.

  Andrea

Reply via email to