diff options
| author | Roman Lebedev <lebedev.ri@gmail.com> | 2017-09-07 22:14:25 +0000 |
|---|---|---|
| committer | Roman Lebedev <lebedev.ri@gmail.com> | 2017-09-07 22:14:25 +0000 |
| commit | 6aa34aadd162093fe3aada3d3724ea30a143b3db (patch) | |
| tree | a42d678a9da4ff90fbc7ba0b74e9eb8fa1e34c17 /llvm/lib/Bitcode/Reader/BitcodeReader.cpp | |
| parent | f26c54f9ce45802213ebb7dd49631c9fa998594c (diff) | |
| download | bcm5719-llvm-6aa34aadd162093fe3aada3d3724ea30a143b3db.tar.gz bcm5719-llvm-6aa34aadd162093fe3aada3d3724ea30a143b3db.zip | |
[Sema] -Wtautological-compare: handle comparison of unsigned with 0S.
Summary:
This is a first half(?) of a fix for the following bug:
https://bugs.llvm.org/show_bug.cgi?id=34147 (gcc -Wtype-limits)
GCC's -Wtype-limits does warn on comparison of unsigned value
with signed zero (as in, with 0), but clang only warns if the
zero is unsigned (i.e. 0U).
Also, be careful not to double-warn, or falsely warn on
comparison of signed/fp variable and signed 0.
Yes, all these testcases are needed.
Testing: $ ninja check-clang-sema check-clang-semacxx
Also, no new warnings for clang stage-2 build.
Reviewers: rjmccall, rsmith, aaron.ballman
Reviewed By: rjmccall
Subscribers: cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D37565
llvm-svn: 312750
Diffstat (limited to 'llvm/lib/Bitcode/Reader/BitcodeReader.cpp')
0 files changed, 0 insertions, 0 deletions
