[PATCH] D49999: [Hexagon] Remove fp-contract=fast setting for at O3

[Brendon Cahoon via Phabricator via cfe-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rC342078: [Hexagon] Remove fp-contract=fast setting for at O3 (authored by bcahoon, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D49999?vs=158029&id=165143#toc

Repository:
  rC Clang

https://reviews.llvm.org/D49999

Files:
  lib/Driver/ToolChains/Hexagon.cpp
  test/Driver/hexagon-toolchain-elf.c


Index: lib/Driver/ToolChains/Hexagon.cpp
===================================================================
--- lib/Driver/ToolChains/Hexagon.cpp
+++ lib/Driver/ToolChains/Hexagon.cpp
@@ -513,11 +513,6 @@
 void HexagonToolChain::addClangTargetOptions(const ArgList &DriverArgs,
                                              ArgStringList &CC1Args,
                                              Action::OffloadKind) const {
-  if (!DriverArgs.hasArg(options::OPT_ffp_contract)) {
-    unsigned OptLevel = getOptimizationLevel(DriverArgs);
-    if (OptLevel >= 3)
-      CC1Args.push_back("-ffp-contract=fast");
-  }
   if (DriverArgs.hasArg(options::OPT_ffixed_r19)) {
     CC1Args.push_back("-target-feature");
     CC1Args.push_back("+reserved-r19");
Index: test/Driver/hexagon-toolchain-elf.c
===================================================================
--- test/Driver/hexagon-toolchain-elf.c
+++ test/Driver/hexagon-toolchain-elf.c
@@ -110,7 +110,7 @@
 // RUN:   -O3 \
 // RUN:   %s 2>&1 \
 // RUN:   | FileCheck -check-prefix=CHECK026 %s
-// CHECK026: "-ffp-contract=fast"
+// CHECK026-NOT: "-ffp-contract=fast"
 // CHECK026: hexagon-link
 
 // RUN: %clang -### -target hexagon-unknown-elf \


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D49999.165143.patch
Type: text/x-patch
Size: 1205 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180912/a2ff598c/attachment.bin>