[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