[clang] 5dcffdf - Fix fp-model flag test failure on linux
Matt Arsenault via cfe-commits
cfe-commits at lists.llvm.org
Wed Feb 12 10:14:40 PST 2020
Author: Matt Arsenault
Date: 2020-02-12T10:14:29-08:00
New Revision: 5dcffdf58a33b51028f6b52e44c4af25b5d159aa
URL: https://github.com/llvm/llvm-project/commit/5dcffdf58a33b51028f6b52e44c4af25b5d159aa
DIFF: https://github.com/llvm/llvm-project/commit/5dcffdf58a33b51028f6b52e44c4af25b5d159aa.diff
LOG: Fix fp-model flag test failure on linux
We're still in the awkward state where IEEE is not the default
denormal mode.
Added:
Modified:
clang/lib/Driver/ToolChains/Clang.cpp
Removed:
################################################################################
diff --git a/clang/lib/Driver/ToolChains/Clang.cpp b/clang/lib/Driver/ToolChains/Clang.cpp
index a11a5423b0b9..79a518c2904c 100644
--- a/clang/lib/Driver/ToolChains/Clang.cpp
+++ b/clang/lib/Driver/ToolChains/Clang.cpp
@@ -2771,7 +2771,8 @@ static void RenderFloatingPointOptions(const ToolChain &TC, const Driver &D,
if (HonorINFs && HonorNaNs &&
!AssociativeMath && !ReciprocalMath &&
SignedZeros && TrappingMath && RoundingFPMath &&
- DenormalFPMath != llvm::DenormalMode::getIEEE() &&
+ // FIXME: This should check for IEEE when it's the default.
+ DenormalFPMath != llvm::DenormalMode::getInvalid() &&
FPContract.equals("off"))
// OK: Current Arg doesn't conflict with -ffp-model=strict
;
More information about the cfe-commits
mailing list