rob.lougher added a comment.
Ping. Just test changes - OK to commit?
http://reviews.llvm.org/D16298
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rob.lougher added a comment.
Ping.
http://reviews.llvm.org/D16298
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rob.lougher added a comment.
No problem. Thanks for the code review.
http://reviews.llvm.org/D16298
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: rlougher
Date: Fri Apr 29 12:44:29 2016
New Revision: 268075
URL: http://llvm.org/viewvc/llvm-project?rev=268075&view=rev
Log:
Improve test coverage of -Wdouble-promotion
This patch adds coverage for additional cases where implicit conversion can
occur (assignment and return). It also ad
rob.lougher added a comment.
Ping. Please can somebody review this? Thanks!
http://reviews.llvm.org/D16298
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rob.lougher created this revision.
rob.lougher added a reviewer: gbiv.
rob.lougher added a subscriber: cfe-commits.
The -Wdouble-promotion warning was added to clang in r251588 (fixing PR15109).
The added test, however, only covered a subset of the cases where implicit
conversion can occur (ass
rob.lougher added a comment.
Ping.
http://reviews.llvm.org/D16298
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
