This revision was automatically updated to reflect the committed changes.
Closed by commit rL340424: [Android] Default to -fno-math-errno (authored by
pirama, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews.llvm.org/D51068
Files:
cfe/trunk/lib/Dr
srhines accepted this revision.
srhines added inline comments.
This revision is now accepted and ready to land.
Comment at: lib/Driver/ToolChains/Linux.cpp:913
+return false;
+ return Generic_ELF::IsMathErrnoDefault();
+}
pirama wrote:
> I tried to be defens
pirama added inline comments.
Comment at: lib/Driver/ToolChains/Linux.cpp:913
+return false;
+ return Generic_ELF::IsMathErrnoDefault();
+}
I tried to be defensive here in case the default changes in the future. I can
simplify to just return true here if t
pirama created this revision.
pirama added reviewers: srhines, enh.
Android's libm does not set errno.
Repository:
rC Clang
https://reviews.llvm.org/D51068
Files:
lib/Driver/ToolChains/Linux.cpp
lib/Driver/ToolChains/Linux.h
test/Driver/fast-math.c
Index: test/Driver/fast-math.c