summaryrefslogtreecommitdiffstats
path: root/llvm/lib
diff options
context:
space:
mode:
authorKirill Bobyrev <omtcyfz@gmail.com>2016-08-10 18:01:45 +0000
committerKirill Bobyrev <omtcyfz@gmail.com>2016-08-10 18:01:45 +0000
commit8694cb97c2c1aead7e8a450659a25bc1c95943ab (patch)
tree6005cd43d232e74e7a990518dc3789873c5b28a1 /llvm/lib
parentd00efc6795f8445c1fd3a1112115ca39461cef8e (diff)
downloadbcm5719-llvm-8694cb97c2c1aead7e8a450659a25bc1c95943ab.tar.gz
bcm5719-llvm-8694cb97c2c1aead7e8a450659a25bc1c95943ab.zip
[clang-tidy] minor improvements in modernise-deprecated-headers check
This patch introduces a minor list of changes as proposed by Richard Smith in the mailing list. See original comments with an impact on the future check state below: [comments.begin > + {"complex.h", "ccomplex"}, It'd be better to convert this one to <complex>, or leave it alone. <ccomplex> is an unnecessary wart. (The contents of C++11's <complex.h> / <ccomplex> / <complex> (all of which are identical) aren't comparable to C99's <complex.h>, so if this was C++98 code using the C99 header, the code will be broken with or without this transformation.) > + {"iso646.h", "ciso646"}, Just delete #includes of this one. <ciso646> does nothing. > + {"stdalign.h", "cstdalign"}, > + {"stdbool.h", "cstdbool"}, We should just delete these two includes. These headers do nothing in C++. comments.end] Reviewers: alexfh, aaron.ballman Differential Revision: https://reviews.llvm.org/D17990 llvm-svn: 278254
Diffstat (limited to 'llvm/lib')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud