From: Chandler Carruth Date: Fri, 17 Jun 2016 07:15:29 +0000 (+0000) Subject: [PM] Run clang-format over various parts of the new pass manager code X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=bd6882c6771b8b09b1aacd7c28686cdaf252f7df;p=llvm [PM] Run clang-format over various parts of the new pass manager code prior to some very substantial patches to isolate any formatting-only changes. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@272991 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/include/llvm/IR/PassManagerInternal.h b/include/llvm/IR/PassManagerInternal.h index 0c619e8c89e..4351b588828 100644 --- a/include/llvm/IR/PassManagerInternal.h +++ b/include/llvm/IR/PassManagerInternal.h @@ -18,8 +18,8 @@ #ifndef LLVM_IR_PASSMANAGERINTERNAL_H #define LLVM_IR_PASSMANAGERINTERNAL_H -#include "llvm/ADT/StringRef.h" #include "llvm/ADT/STLExtras.h" +#include "llvm/ADT/StringRef.h" namespace llvm { diff --git a/lib/IR/PassManager.cpp b/lib/IR/PassManager.cpp index 9d5b4cb0516..8563a4019a2 100644 --- a/lib/IR/PassManager.cpp +++ b/lib/IR/PassManager.cpp @@ -7,9 +7,9 @@ // //===----------------------------------------------------------------------===// +#include "llvm/IR/PassManager.h" #include "llvm/ADT/STLExtras.h" #include "llvm/IR/LLVMContext.h" -#include "llvm/IR/PassManager.h" using namespace llvm; diff --git a/lib/Passes/PassBuilder.cpp b/lib/Passes/PassBuilder.cpp index 9af7862b16d..eb0da0953c0 100644 --- a/lib/Passes/PassBuilder.cpp +++ b/lib/Passes/PassBuilder.cpp @@ -86,11 +86,11 @@ #include "llvm/Transforms/Scalar/PartiallyInlineLibCalls.h" #include "llvm/Transforms/Scalar/Reassociate.h" #include "llvm/Transforms/Scalar/SCCP.h" +#include "llvm/Transforms/Scalar/SLPVectorizer.h" #include "llvm/Transforms/Scalar/SROA.h" #include "llvm/Transforms/Scalar/SimplifyCFG.h" #include "llvm/Transforms/Scalar/Sink.h" #include "llvm/Transforms/Utils/AddDiscriminators.h" -#include "llvm/Transforms/Scalar/SLPVectorizer.h" #include "llvm/Transforms/Utils/LCSSA.h" #include "llvm/Transforms/Utils/Mem2Reg.h" #include "llvm/Transforms/Utils/MemorySSA.h" @@ -190,25 +190,25 @@ char NoOpLoopAnalysis::PassID; } // End anonymous namespace. void PassBuilder::registerModuleAnalyses(ModuleAnalysisManager &MAM) { -#define MODULE_ANALYSIS(NAME, CREATE_PASS) \ +#define MODULE_ANALYSIS(NAME, CREATE_PASS) \ MAM.registerPass([&] { return CREATE_PASS; }); #include "PassRegistry.def" } void PassBuilder::registerCGSCCAnalyses(CGSCCAnalysisManager &CGAM) { -#define CGSCC_ANALYSIS(NAME, CREATE_PASS) \ +#define CGSCC_ANALYSIS(NAME, CREATE_PASS) \ CGAM.registerPass([&] { return CREATE_PASS; }); #include "PassRegistry.def" } void PassBuilder::registerFunctionAnalyses(FunctionAnalysisManager &FAM) { -#define FUNCTION_ANALYSIS(NAME, CREATE_PASS) \ +#define FUNCTION_ANALYSIS(NAME, CREATE_PASS) \ FAM.registerPass([&] { return CREATE_PASS; }); #include "PassRegistry.def" } void PassBuilder::registerLoopAnalyses(LoopAnalysisManager &LAM) { -#define LOOP_ANALYSIS(NAME, CREATE_PASS) \ +#define LOOP_ANALYSIS(NAME, CREATE_PASS) \ LAM.registerPass([&] { return CREATE_PASS; }); #include "PassRegistry.def" } @@ -250,7 +250,9 @@ static bool isModulePassName(StringRef Name) { if (Name.startswith("default") || Name.startswith("lto")) return DefaultAliasRegex.match(Name); -#define MODULE_PASS(NAME, CREATE_PASS) if (Name == NAME) return true; +#define MODULE_PASS(NAME, CREATE_PASS) \ + if (Name == NAME) \ + return true; #define MODULE_ANALYSIS(NAME, CREATE_PASS) \ if (Name == "require<" NAME ">" || Name == "invalidate<" NAME ">") \ return true; @@ -261,7 +263,9 @@ static bool isModulePassName(StringRef Name) { #endif static bool isCGSCCPassName(StringRef Name) { -#define CGSCC_PASS(NAME, CREATE_PASS) if (Name == NAME) return true; +#define CGSCC_PASS(NAME, CREATE_PASS) \ + if (Name == NAME) \ + return true; #define CGSCC_ANALYSIS(NAME, CREATE_PASS) \ if (Name == "require<" NAME ">" || Name == "invalidate<" NAME ">") \ return true; @@ -271,7 +275,9 @@ static bool isCGSCCPassName(StringRef Name) { } static bool isFunctionPassName(StringRef Name) { -#define FUNCTION_PASS(NAME, CREATE_PASS) if (Name == NAME) return true; +#define FUNCTION_PASS(NAME, CREATE_PASS) \ + if (Name == NAME) \ + return true; #define FUNCTION_ANALYSIS(NAME, CREATE_PASS) \ if (Name == "require<" NAME ">" || Name == "invalidate<" NAME ">") \ return true; @@ -281,7 +287,9 @@ static bool isFunctionPassName(StringRef Name) { } static bool isLoopPassName(StringRef Name) { -#define LOOP_PASS(NAME, CREATE_PASS) if (Name == NAME) return true; +#define LOOP_PASS(NAME, CREATE_PASS) \ + if (Name == NAME) \ + return true; #define LOOP_ANALYSIS(NAME, CREATE_PASS) \ if (Name == "require<" NAME ">" || Name == "invalidate<" NAME ">") \ return true; @@ -384,8 +392,7 @@ bool PassBuilder::parseFunctionPassName(FunctionPassManager &FPM, return false; } -bool PassBuilder::parseLoopPassName(LoopPassManager &FPM, - StringRef Name) { +bool PassBuilder::parseLoopPassName(LoopPassManager &FPM, StringRef Name) { #define LOOP_PASS(NAME, CREATE_PASS) \ if (Name == NAME) { \ FPM.addPass(CREATE_PASS); \ @@ -707,7 +714,6 @@ bool PassBuilder::parsePassPipeline(ModulePassManager &MPM, return true; } - return false; } diff --git a/unittests/Analysis/CGSCCPassManagerTest.cpp b/unittests/Analysis/CGSCCPassManagerTest.cpp index 431fb62cdc9..bc2601565d3 100644 --- a/unittests/Analysis/CGSCCPassManagerTest.cpp +++ b/unittests/Analysis/CGSCCPassManagerTest.cpp @@ -142,9 +142,8 @@ struct TestModulePass { struct TestSCCPass { TestSCCPass(int &RunCount, int &AnalyzedInstrCount, - int &AnalyzedSCCFunctionCount, - int &AnalyzedModuleFunctionCount, - bool OnlyUseCachedResults = false) + int &AnalyzedSCCFunctionCount, int &AnalyzedModuleFunctionCount, + bool OnlyUseCachedResults = false) : RunCount(RunCount), AnalyzedInstrCount(AnalyzedInstrCount), AnalyzedSCCFunctionCount(AnalyzedSCCFunctionCount), AnalyzedModuleFunctionCount(AnalyzedModuleFunctionCount), diff --git a/unittests/IR/PassManagerTest.cpp b/unittests/IR/PassManagerTest.cpp index 80ed723ae08..c2ac863260a 100644 --- a/unittests/IR/PassManagerTest.cpp +++ b/unittests/IR/PassManagerTest.cpp @@ -238,8 +238,8 @@ TEST_F(PassManagerTest, Basic) { { // Pointless scope to test move assignment. FunctionPassManager NestedFPM; - NestedFPM.addPass(TestFunctionPass(FunctionPassRunCount1, AnalyzedInstrCount1, - AnalyzedFunctionCount1)); + NestedFPM.addPass(TestFunctionPass( + FunctionPassRunCount1, AnalyzedInstrCount1, AnalyzedFunctionCount1)); FPM = std::move(NestedFPM); } NestedMPM.addPass(createModuleToFunctionPassAdaptor(std::move(FPM)));