This revision was automatically updated to reflect the committed changes.
Closed by commit rL284121: Guard flag –fdenormal-fp-math with –fno-fast-math.
(authored by SjoerdMeijer).
Changed prior to commit:
https://reviews.llvm.org/D25479?vs=74363&id=74508#toc
Repository:
rL LLVM
https://revi
SjoerdMeijer added a comment.
thanks again for reviewing.
https://reviews.llvm.org/D25479
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rengolin accepted this revision.
rengolin added a comment.
This revision is now accepted and ready to land.
LGTM, thanks!
https://reviews.llvm.org/D25479
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/l
SjoerdMeijer updated this revision to Diff 74363.
SjoerdMeijer added a comment.
Thanks for catching this: the logic and test have been updated.
https://reviews.llvm.org/D25479
Files:
lib/Driver/Tools.cpp
test/Driver/denormal-fp-math.c
Index: test/Driver/denormal-fp-math.c
rengolin added inline comments.
Comment at: test/Driver/denormal-fp-math.c:4
// RUN: %clang -### -target arm-unknown-linux-gnu -c %s
-fdenormal-fp-math=positive-zero -v 2>&1 | FileCheck -check-prefix=CHECK-PZ %s
+// RUN: %clang -### -target arm-unknown-linux-gnu -c %s
-fdenorm
SjoerdMeijer created this revision.
SjoerdMeijer added reviewers: rengolin, jmolloy.
SjoerdMeijer added a subscriber: cfe-commits.
Guard the -fdenormal-fp-math with -fno-fast-math. This allows disabling the FP
options with just one flag: –fno-fast-math.
https://reviews.llvm.org/D25479
Files: