[clang] 3833316 - Completely ignore strict FP model and denormal-fp-math interaction

[Matt Arsenault via cfe-commits]

Author: Matt Arsenault
Date: 2020-02-12T13:26:46-05:00
New Revision: 38333164fc7dfc9d4cbf93e443e74809022ed78e

URL: https://github.com/llvm/llvm-project/commit/38333164fc7dfc9d4cbf93e443e74809022ed78e
DIFF: https://github.com/llvm/llvm-project/commit/38333164fc7dfc9d4cbf93e443e74809022ed78e.diff

LOG: Completely ignore strict FP model and denormal-fp-math interaction

No behavior is going to make sense here until the default is IEEE.

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 79a518c2904c..e71fbae5ae77 100644
--- a/clang/lib/Driver/ToolChains/Clang.cpp
+++ b/clang/lib/Driver/ToolChains/Clang.cpp
@@ -2771,8 +2771,6 @@ static void RenderFloatingPointOptions(const ToolChain &TC, const Driver &D,
       if (HonorINFs && HonorNaNs &&
         !AssociativeMath && !ReciprocalMath &&
         SignedZeros && TrappingMath && RoundingFPMath &&
-        // 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
         ;