mirror of
https://github.com/Gericom/teak-llvm.git
synced 2025-06-28 15:58:57 -04:00
8dedeb0230
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
70 lines
2.6 KiB
C++
70 lines
2.6 KiB
C++
//===--- ModernizeTidyModule.cpp - clang-tidy -----------------------------===//
|
|
//
|
|
// The LLVM Compiler Infrastructure
|
|
//
|
|
// This file is distributed under the University of Illinois Open Source
|
|
// License. See LICENSE.TXT for details.
|
|
//
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
#include "../ClangTidy.h"
|
|
#include "../ClangTidyModule.h"
|
|
#include "../ClangTidyModuleRegistry.h"
|
|
#include "LoopConvertCheck.h"
|
|
#include "MakeUniqueCheck.h"
|
|
#include "PassByValueCheck.h"
|
|
#include "ReplaceAutoPtrCheck.h"
|
|
#include "ShrinkToFitCheck.h"
|
|
#include "UseAutoCheck.h"
|
|
#include "UseDefaultCheck.h"
|
|
#include "UseNullptrCheck.h"
|
|
#include "UseOverrideCheck.h"
|
|
|
|
using namespace clang::ast_matchers;
|
|
|
|
namespace clang {
|
|
namespace tidy {
|
|
namespace modernize {
|
|
|
|
class ModernizeModule : public ClangTidyModule {
|
|
public:
|
|
void addCheckFactories(ClangTidyCheckFactories &CheckFactories) override {
|
|
CheckFactories.registerCheck<LoopConvertCheck>("modernize-loop-convert");
|
|
CheckFactories.registerCheck<MakeUniqueCheck>("modernize-make-unique");
|
|
CheckFactories.registerCheck<PassByValueCheck>("modernize-pass-by-value");
|
|
CheckFactories.registerCheck<ReplaceAutoPtrCheck>(
|
|
"modernize-replace-auto-ptr");
|
|
CheckFactories.registerCheck<ShrinkToFitCheck>("modernize-shrink-to-fit");
|
|
CheckFactories.registerCheck<UseAutoCheck>("modernize-use-auto");
|
|
CheckFactories.registerCheck<UseDefaultCheck>("modernize-use-default");
|
|
CheckFactories.registerCheck<UseNullptrCheck>("modernize-use-nullptr");
|
|
CheckFactories.registerCheck<UseOverrideCheck>("modernize-use-override");
|
|
}
|
|
|
|
ClangTidyOptions getModuleOptions() override {
|
|
ClangTidyOptions Options;
|
|
auto &Opts = Options.CheckOptions;
|
|
Opts["modernize-loop-convert.MinConfidence"] = "reasonable";
|
|
Opts["modernize-loop-convert.NamingStyle"] = "CamelCase";
|
|
Opts["modernize-pass-by-value.IncludeStyle"] = "llvm"; // Also: "google".
|
|
Opts["modernize-replace-auto-ptr.IncludeStyle"] = "llvm"; // Also: "google".
|
|
|
|
// Comma-separated list of macros that behave like NULL.
|
|
Opts["modernize-use-nullptr.NullMacros"] = "NULL";
|
|
return Options;
|
|
}
|
|
};
|
|
|
|
// Register the ModernizeTidyModule using this statically initialized variable.
|
|
static ClangTidyModuleRegistry::Add<ModernizeModule> X("modernize-module",
|
|
"Add modernize checks.");
|
|
|
|
} // namespace modernize
|
|
|
|
// This anchor is used to force the linker to link in the generated object file
|
|
// and thus register the ModernizeModule.
|
|
volatile int ModernizeModuleAnchorSource = 0;
|
|
|
|
} // namespace tidy
|
|
} // namespace clang
|