[clang-tidy] Refactor: Move llvm clang-tidy checks to namespace clang::tidy::llvm

clang-tidy checks are organized into modules. This refactoring moves the llvm
module checks to clang::tidy::llvm

http://reviews.llvm.org/D7995

Patch by Richard Thomson!

llvm-svn: 230952

clang-tools-extra/clang-tidy/llvm/HeaderGuardCheck.cpp

@@ -11,6 +11,7 @@
 
 namespace clang {
 namespace tidy {
+namespace llvm {
 
 bool LLVMHeaderGuardCheck::shouldFixHeaderGuard(StringRef Filename) {
   return Filename.endswith(".h");
@@ -49,5 +50,6 @@ std::string LLVMHeaderGuardCheck::getHeaderGuard(StringRef Filename,
   return StringRef(Guard).upper();
 }
 
+} // namespace llvm
 } // namespace tidy
 } // namespace clang

clang-tools-extra/clang-tidy/llvm/HeaderGuardCheck.h

@@ -14,6 +14,7 @@
 
 namespace clang {
 namespace tidy {
+namespace llvm {
 
 /// Finds and fixes header guards that do not adhere to LLVM style.
 class LLVMHeaderGuardCheck : public HeaderGuardCheck {
@@ -25,6 +26,7 @@ public:
   std::string getHeaderGuard(StringRef Filename, StringRef OldGuard) override;
 };
 
+} // namespace llvm
 } // namespace tidy
 } // namespace clang
 

clang-tools-extra/clang-tidy/llvm/IncludeOrderCheck.cpp

@@ -14,6 +14,7 @@
 
 namespace clang {
 namespace tidy {
+namespace llvm {
 
 namespace {
 class IncludeOrderPPCallbacks : public PPCallbacks {
@@ -46,8 +47,8 @@ private:
 
 void IncludeOrderCheck::registerPPCallbacks(CompilerInstance &Compiler) {
   Compiler.getPreprocessor().addPPCallbacks(
-      llvm::make_unique<IncludeOrderPPCallbacks>(*this,
-                                                 Compiler.getSourceManager()));
+      ::llvm::make_unique<IncludeOrderPPCallbacks>(
+          *this, Compiler.getSourceManager()));
 }
 
 static int getPriority(StringRef Filename, bool IsAngled, bool IsMainModule) {
@@ -163,5 +164,6 @@ void IncludeOrderPPCallbacks::EndOfMainFile() {
   IncludeDirectives.clear();
 }
 
+} // namespace llvm
 } // namespace tidy
 } // namespace clang

clang-tools-extra/clang-tidy/llvm/IncludeOrderCheck.h

@@ -14,6 +14,7 @@
 
 namespace clang {
 namespace tidy {
+namespace llvm {
 
 /// \brief Checks the correct order of \c #includes.
 ///
@@ -25,6 +26,7 @@ public:
   void registerPPCallbacks(CompilerInstance &Compiler) override;
 };
 
+} // namespace llvm
 } // namespace tidy
 } // namespace clang
 

clang-tools-extra/clang-tidy/llvm/LLVMTidyModule.cpp

@@ -17,6 +17,7 @@
 
 namespace clang {
 namespace tidy {
+namespace llvm {
 
 class LLVMModule : public ClangTidyModule {
 public:
@@ -33,6 +34,8 @@ public:
 static ClangTidyModuleRegistry::Add<LLVMModule> X("llvm-module",
                                                   "Adds LLVM lint checks.");
 
+} // namespace llvm
+
 // This anchor is used to force the linker to link in the generated object file
 // and thus register the LLVMModule.
 volatile int LLVMModuleAnchorSource = 0;

clang-tools-extra/clang-tidy/llvm/TwineLocalCheck.cpp

@@ -16,6 +16,7 @@ using namespace clang::ast_matchers;
 
 namespace clang {
 namespace tidy {
+namespace llvm {
 
 void TwineLocalCheck::registerMatchers(MatchFinder *Finder) {
   auto TwineType =
@@ -57,5 +58,6 @@ void TwineLocalCheck::check(const MatchFinder::MatchResult &Result) {
   }
 }
 
+} // namespace llvm
 } // namespace tidy
 } // namespace clang

clang-tools-extra/clang-tidy/llvm/TwineLocalCheck.h

@@ -14,6 +14,7 @@
 
 namespace clang {
 namespace tidy {
+namespace llvm {
 
 /// \brief Looks for local Twine variables which are prone to use after frees
 /// and should be generally avoided.
@@ -25,6 +26,7 @@ public:
   void check(const ast_matchers::MatchFinder::MatchResult &Result) override;
 };
 
+} // namespace llvm
 } // namespace tidy
 } // namespace clang
 

clang-tools-extra/unittests/clang-tidy/LLVMModuleTest.cpp

@@ -3,6 +3,8 @@
 #include "llvm/IncludeOrderCheck.h"
 #include "gtest/gtest.h"
 
+using namespace clang::tidy::llvm;
+
 namespace clang {
 namespace tidy {
 namespace test {