Skip to content
This repository was archived by the owner on Sep 2, 2018. It is now read-only.

Commit 1610d6e

Browse files
author
Charlie Turner
committed
Add missing FP build attribute tests.
The test file test/CodeGen/ARM/build-attributes.ll was missing several floating-point build attribute tests. The intention of this commit is that for each CPU / architecture currently tested, there are now tests that make sure the following attributes are sufficiently checked, * Tag_ABI_FP_rounding * Tag_ABI_FP_denormal * Tag_ABI_FP_exceptions * Tag_ABI_FP_user_exceptions * Tag_ABI_FP_number_model Also in this commit, the -unsafe-fp-math flag has been augmented with the full suite of flags Clang sends to LLVM when you pass -ffast-math to Clang. That is, `-unsafe-fp-math' has been changed to `-enable-unsafe-fp-math -disable-fp-elim -enable-no-infs-fp-math -enable-no-nans-fp-math -fp-contract=fast' Change-Id: I35d766076bcbbf09021021c0a534bf8bf9a32dfc git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@223454 91177308-0d34-0410-b5e6-96231b3b80d8
1 parent 138d5bf commit 1610d6e

File tree

2 files changed

+150
-28
lines changed

2 files changed

+150
-28
lines changed

lib/Target/ARM/ARMAsmPrinter.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -747,6 +747,8 @@ void ARMAsmPrinter::emitAttributes() {
747747
// absence of its emission implies zero).
748748
}
749749

750+
// TM.Options.NoInfsFPMath && TM.Options.NoNaNsFPMath is the
751+
// equivalent of GCC's -ffinite-math-only flag.
750752
if (TM.Options.NoInfsFPMath && TM.Options.NoNaNsFPMath)
751753
ATS.emitAttribute(ARMBuildAttrs::ABI_FP_number_model,
752754
ARMBuildAttrs::Allowed);

0 commit comments

Comments
 (0)