summaryrefslogtreecommitdiffstats
path: root/clang-tools-extra/clang-tidy/modernize/UseDefaultCheck.cpp
Commit message (Collapse)AuthorAgeFilesLines
* Make modernize-use-default tolerant of delayed template parsing; this code ↵Aaron Ballman2016-01-201-0/+1
| | | | | | was previously causing failed assertions because dyn_cast was being passed a null pointer due to the delay-parsed body. llvm-svn: 258356
* Replace a dyn_cast with isa where the result was only being used as a ↵Craig Topper2015-11-181-1/+1
| | | | | | boolean. NFC. llvm-svn: 253442
* modernize-use-default supports copy constructor and copy-assignment operator.Angel Garcia Gomez2015-11-021-24/+282
| | | | | | | | | | | | Summary: the check will now warn when the user provided definitions of this functions is equivalent to the explicitly defaulted ones. Reviewers: klimek Subscribers: klimek, cfe-commits, alexfh Differential Revision: http://reviews.llvm.org/D14145 llvm-svn: 251788
* Add modernize-use-default check to clang-tidy.Angel Garcia Gomez2015-10-211-0/+63
Summary: Add a check that replaces empty bodies of special member functions with '= default;'. For now, it is only implemented for the default constructor and the destructor, which are the easier cases. The copy-constructor and the copy-assignment operator cases will be implemented later. I applied this check to the llvm code base and found 627 warnings (385 in llvm, 9 in compiler-rt, 220 in clang and 13 in clang-tools-extra). Applying the fixes didn't break any build or test, it only caused a -Wpedantic warning in lib/Target/Mips/MipsOptionRecord.h:33 becaused it replaced virtual ~MipsOptionRecord(){}; to virtual ~MipsOptionRecord()= default;; Reviewers: klimek Subscribers: george.burgess.iv, Eugene.Zelenko, alexfh, cfe-commits Differential Revision: http://reviews.llvm.org/D13871 llvm-svn: 250897
OpenPOWER on IntegriCloud