[PATCH] D138675: [flang] Add -ffast-math and -Ofast

[Tom Eccles via Phabricator via cfe-commits]

tblah marked an inline comment as done.
tblah added inline comments.


================
Comment at: flang/test/Driver/fast_math.f90:59-62
+! Check that -ffast-math causes us to link to crtfastmath.o
+! UNSUPPORTED: system-windows
+! RUN: %flang -ffast-math -### %s -o %t 2>&1 \
+! RUN:     | FileCheck --check-prefix=CHECK-CRT %s
----------------
mnadeem wrote:
> I think this test should use an explicit sysroot like the clang test does. It fails on my system (x86 linux, llvm built for arm target) because it cant find the crtfastmath.o library.
> 
> 
> > // file linux-ld.c
> >
> > // RUN:        --sysroot=%S/Inputs/basic_linux_tree 2>&1 \
> > // RUN:   | FileCheck --check-prefix=CHECK-CRTFASTMATH %s
> 
Thanks for the suggestion https://reviews.llvm.org/D142807


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D138675/new/

https://reviews.llvm.org/D138675