This revision was automatically updated to reflect the committed changes.
Closed by commit rG5830da1f8625: [AArch64] Define __FP_FAST_FMA[F] (authored by
jolanta.jensen).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D127655/new/
jolanta.jensen updated this revision to Diff 438344.
jolanta.jensen added a comment.
Removing an unnecessary test change from aarch64-target-features.c
as init-aarch64.c provides enough coverage.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
paulwalker-arm added inline comments.
Comment at: clang/test/Preprocessor/aarch64-target-features.c:59-60
// CHECK-NOT: __ARM_FEATURE_SVE_BITS 2048
+// CHECK: __FP_FAST_FMA 1
+// CHECK: __FP_FAST_FMAF 1
I don't think we need this change given `init-aarch64.c`
jolanta.jensen created this revision.
Herald added a subscriber: kristof.beyls.
Herald added a project: All.
jolanta.jensen requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Libraries use this flag to decide whether to use the fma builtin.