Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Make modernize-use-default tolerant of delayed template parsing; this code ↵ | Aaron Ballman | 2016-01-20 | 1 | -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 Topper | 2015-11-18 | 1 | -1/+1 |
| | | | | | | boolean. NFC. llvm-svn: 253442 | ||||
* | modernize-use-default supports copy constructor and copy-assignment operator. | Angel Garcia Gomez | 2015-11-02 | 1 | -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 Gomez | 2015-10-21 | 1 | -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 |