[PATCH] D25479: Guard flag –fdenormal-fp-math with –fno-fast-math

2016-10-13 Thread Sjoerd Meijer via cfe-commits
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

[PATCH] D25479: Guard flag –fdenormal-fp-math with –fno-fast-math

2016-10-13 Thread Sjoerd Meijer via cfe-commits
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

[PATCH] D25479: Guard flag –fdenormal-fp-math with –fno-fast-math

2016-10-13 Thread Renato Golin via 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

[PATCH] D25479: Guard flag –fdenormal-fp-math with –fno-fast-math

2016-10-12 Thread Sjoerd Meijer via cfe-commits
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

[PATCH] D25479: Guard flag –fdenormal-fp-math with –fno-fast-math

2016-10-12 Thread Renato Golin via cfe-commits
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

[PATCH] D25479: Guard flag –fdenormal-fp-math with –fno-fast-math

2016-10-11 Thread Sjoerd Meijer via cfe-commits
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: