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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to