> On 3 Jul 2024, at 11:59, Kyrylo Tkachov <ktkac...@nvidia.com> wrote:
> 
> External email: Use caution opening links or attachments
> 
> 
> Hi all,
> 
> The ACLE asks the user to test for __ARM_FEATURE_BF16 before using the
> <arm_bf16.h> header but GCC doesn't set this up.
> LLVM does, so this is an inconsistency between the compilers.
> 
> This patch enables that macro for TARGET_BF16_FP.
> Bootstrapped and tested on aarch64-none-linux-gnu.
> Pushing to trunk.
> I think it makes sense to back port this to branches that support the 
> arm_bf16.h header.
> I’ll prepare patches for those separately.
> 
> Thanks,
> Kyrill
> 
> gcc/
> 
>        PR target/115457
>        * config/aarch64/aarch64-c.c (aarch64_update_cpp_builtins):
>        Define __ARM_FEATURE_BF16 for TARGET_BF16_FP.

This should be aarch64-c.cc <http://aarch64-c.cc/> of course. I’ve fixed that 
up on commit and in the second patch too.
Thanks,
Kyrill

> 
> gcc/testsuite/
> 
>        PR target/115457
>        * gcc.target/aarch64/acle/bf16_feature.c: New test.
> 
> Signed-off-by: Kyrylo Tkachov <ktkac...@nvidia.com>
> <0001-aarch64-PR-target-115457-Implement-missing-__ARM_FEA.patch>

Reply via email to