This revision was automatically updated to reflect the committed changes.
Closed by commit rL283109: This adds a separate file for the fp denormal
regression tests. NFC. (authored by SjoerdMeijer).
Changed prior to commit:
https://reviews.llvm.org/D24907?vs=72457&id=73411#toc
Repository:
rL
SjoerdMeijer added a comment.
Yes, excellent, I will rename it to denormal-fp-math.c and commit. Thanks for
reviewing!
https://reviews.llvm.org/D24907
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/lis
hfinkel accepted this revision.
hfinkel added a reviewer: hfinkel.
hfinkel added a comment.
This revision is now accepted and ready to land.
This LGTM, although I find "fast-math.c" much easier to read than
"denormalfpmode.c". How about naming it "denormal-fp-math.c" to match the
option name?
SjoerdMeijer created this revision.
SjoerdMeijer added reviewers: rengolin, jmolloy, olista01.
SjoerdMeijer added a subscriber: cfe-commits.
For clarity, this moves checks for -fdenormal-fp-math values from fast-math.c
to a new separate file.
https://reviews.llvm.org/D24907
Files:
test/Driver
