[libc-commits] [PATCH] D123615: [libc] Automatically add -mfma flag for architectures supporting FMA.
Tue Ly via Phabricator via libc-commits
libc-commits at lists.llvm.org
Wed Jun 1 06:09:23 PDT 2022
lntue added inline comments.
Comment at: libc/test/src/math/CMakeLists.txt:1204-1205
> I'm not sure I understand the semantic here. Does that mean that the test requires FMA?
> If so does it? It would make sense to test it with and without FMA.
> Maybe I missed something.
Yes, for this patch, this test does require FMA. That is because without FMA instructions, the current expm1f implementation is not correctly rounded for all float inputs (1 extra exceptional value). This will be fixed in the followup patch: https://reviews.llvm.org/D123440
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the libc-commits