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.

gcc/testsuite/

        PR target/115457
        * gcc.target/aarch64/acle/bf16_feature.c: New test.

Signed-off-by: Kyrylo Tkachov <ktkac...@nvidia.com>

Attachment: 0001-aarch64-PR-target-115457-Implement-missing-__ARM_FEA.patch
Description: 0001-aarch64-PR-target-115457-Implement-missing-__ARM_FEA.patch

Reply via email to