From fbb46500cf21ff15c9b0ac18c7f69417bdf7571f Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Sun, 27 Aug 2006 22:21:55 +0000 Subject: [PATCH] We no longer care whether something is an opt vs analysis pass, only whether something is a pass vs an analysis group now. Simplify interfaces. llvm-svn: 29920 --- llvm/include/llvm/PassSupport.h | 70 +++++++++++++-------------------- llvm/lib/VMCore/Pass.cpp | 6 +-- 2 files changed, 31 insertions(+), 45 deletions(-) diff --git a/llvm/include/llvm/PassSupport.h b/llvm/include/llvm/PassSupport.h index c802fe2f0d58..8d978b35d1b8 100644 --- a/llvm/include/llvm/PassSupport.h +++ b/llvm/include/llvm/PassSupport.h @@ -38,28 +38,18 @@ class PassInfo { const char *PassName; // Nice name for Pass const char *PassArgument; // Command Line argument to run this pass const std::type_info &TypeInfo; // type_info object for this Pass class - unsigned char PassType; // Set of enums values below... + bool IsAnalysisGroup; // True if an analysis group. std::vector ItfImpl;// Interfaces implemented by this pass Pass *(*NormalCtor)(); // No argument ctor Pass *(*TargetCtor)(TargetMachine&); // Ctor taking TargetMachine object... public: - /// PassType - Define symbolic constants that can be used to test to see if - /// this pass should be listed by analyze or opt. Passes can use none, one or - /// many of these flags or'd together. It is not legal to combine the - /// AnalysisGroup flag with others. - /// - enum { - Analysis = 1, Optimization = 2, AnalysisGroup = 4 - }; - /// PassInfo ctor - Do not call this directly, this should only be invoked /// through RegisterPass. PassInfo(const char *name, const char *arg, const std::type_info &ti, - unsigned char pt, Pass *(*normal)() = 0, - Pass *(*targetctor)(TargetMachine &) = 0) - : PassName(name), PassArgument(arg), TypeInfo(ti), PassType(pt), + Pass *(*normal)() = 0, Pass *(*targetctor)(TargetMachine &) = 0) + : PassName(name), PassArgument(arg), TypeInfo(ti), IsAnalysisGroup(false), NormalCtor(normal), TargetCtor(targetctor) { } @@ -78,11 +68,11 @@ public: /// const std::type_info &getTypeInfo() const { return TypeInfo; } - /// getPassType - Return the PassType of a pass. Note that this can be - /// several different types or'd together. This is _strictly_ for use by opt, - /// analyze and llc for deciding which passes to use as command line options. + /// isAnalysisGroup - Return true if this is an analysis group, not a normal + /// pass. /// - unsigned getPassType() const { return PassType; } + bool isAnalysisGroup() const { return IsAnalysisGroup; } + void SetIsAnalysisGroup() { IsAnalysisGroup = true; } /// getNormalCtor - Return a pointer to a function, that when called, creates /// an instance of the pass and returns it. This pointer may be null if there @@ -97,7 +87,7 @@ public: /// createPass() - Use this method to create an instance of this pass. Pass *createPass() const { - assert((PassType != AnalysisGroup || NormalCtor) && + assert((!isAnalysisGroup() || NormalCtor) && "No default implementation found for analysis group!"); assert(NormalCtor && "Cannot call createPass on PassInfo without default ctor!"); @@ -153,21 +143,21 @@ struct RegisterPassBase { const PassInfo *getPassInfo() const { return &PIObj; } RegisterPassBase(const char *Name, const char *Arg, const std::type_info &TI, - unsigned char PT, Pass *(*Normal)() = 0, + Pass *(*Normal)() = 0, Pass *(*TargetCtor)(TargetMachine &) = 0) - : PIObj(Name, Arg, TI, PT, Normal, TargetCtor) { + : PIObj(Name, Arg, TI, Normal, TargetCtor) { registerPass(); } - RegisterPassBase(const std::type_info &TI, unsigned char PT) - : PIObj("", "", TI, PT, 0, 0) { + RegisterPassBase(const std::type_info &TI) + : PIObj("", "", TI, 0, 0) { // This ctor may only be used for analysis groups: it does not auto-register // the pass. - assert(PT == PassInfo::AnalysisGroup && "Not an AnalysisGroup!"); + PIObj.SetIsAnalysisGroup(); } ~RegisterPassBase() { // Intentionally non-virtual. // Analysis groups are registered/unregistered by their dtor. - if (PIObj.getPassType() != PassInfo::AnalysisGroup) + if (!PIObj.isAnalysisGroup()) unregisterPass(); } @@ -189,26 +179,24 @@ template struct RegisterPass : public RegisterPassBase { // Register Pass using default constructor... - RegisterPass(const char *PassArg, const char *Name, unsigned char PassTy = 0) - : RegisterPassBase(Name, PassArg, typeid(PassName), PassTy, + RegisterPass(const char *PassArg, const char *Name) + : RegisterPassBase(Name, PassArg, typeid(PassName), callDefaultCtor) {} // Register Pass using default constructor explicitly... - RegisterPass(const char *PassArg, const char *Name, unsigned char PassTy, + RegisterPass(const char *PassArg, const char *Name, Pass *(*ctor)()) - : RegisterPassBase(Name, PassArg, typeid(PassName), PassTy, ctor) {} + : RegisterPassBase(Name, PassArg, typeid(PassName), ctor) {} // Register Pass using TargetMachine constructor... - RegisterPass(const char *PassArg, const char *Name, unsigned char PassTy, + RegisterPass(const char *PassArg, const char *Name, Pass *(*targetctor)(TargetMachine &)) - : RegisterPassBase(Name, PassArg, typeid(PassName), PassTy, - 0, targetctor) {} + : RegisterPassBase(Name, PassArg, typeid(PassName), 0, targetctor) {} // Generic constructor version that has an unknown ctor type... template - RegisterPass(const char *PassArg, const char *Name, unsigned char PassTy, - CtorType *Fn) - : RegisterPassBase(Name, PassArg, typeid(PassName), PassTy, 0) {} + RegisterPass(const char *PassArg, const char *Name, CtorType *Fn) + : RegisterPassBase(Name, PassArg, typeid(PassName), 0) {} }; /// RegisterOpt - Register something that is to show up in Opt, this is just a @@ -217,7 +205,7 @@ struct RegisterPass : public RegisterPassBase { template struct RegisterOpt : public RegisterPassBase { RegisterOpt(const char *PassArg, const char *Name, bool CFGOnly = false) - : RegisterPassBase(Name, PassArg, typeid(PassName), PassInfo::Optimization, + : RegisterPassBase(Name, PassArg, typeid(PassName), callDefaultCtor) { if (CFGOnly) setOnlyUsesCFG(); } @@ -226,8 +214,7 @@ struct RegisterOpt : public RegisterPassBase { /// RegisterOpt(const char *PassArg, const char *Name, Pass *(*ctor)(), bool CFGOnly = false) - : RegisterPassBase(Name, PassArg, typeid(PassName), - PassInfo::Optimization, ctor) { + : RegisterPassBase(Name, PassArg, typeid(PassName), ctor) { if (CFGOnly) setOnlyUsesCFG(); } @@ -235,7 +222,7 @@ struct RegisterOpt : public RegisterPassBase { /// RegisterOpt(const char *PassArg, const char *Name, FunctionPass *(*ctor)(), bool CFGOnly = false) - : RegisterPassBase(Name, PassArg, typeid(PassName), PassInfo::Optimization, + : RegisterPassBase(Name, PassArg, typeid(PassName), static_cast(ctor)) { if (CFGOnly) setOnlyUsesCFG(); } @@ -244,8 +231,7 @@ struct RegisterOpt : public RegisterPassBase { /// RegisterOpt(const char *PassArg, const char *Name, Pass *(*targetctor)(TargetMachine &), bool CFGOnly = false) - : RegisterPassBase(Name, PassArg, typeid(PassName), - PassInfo::Optimization, 0, targetctor) { + : RegisterPassBase(Name, PassArg, typeid(PassName), 0, targetctor) { if (CFGOnly) setOnlyUsesCFG(); } @@ -254,7 +240,7 @@ struct RegisterOpt : public RegisterPassBase { RegisterOpt(const char *PassArg, const char *Name, FunctionPass *(*targetctor)(TargetMachine &), bool CFGOnly = false) - : RegisterPassBase(Name, PassArg, typeid(PassName), PassInfo::Optimization, 0, + : RegisterPassBase(Name, PassArg, typeid(PassName), 0, static_cast(targetctor)) { if (CFGOnly) setOnlyUsesCFG(); } @@ -270,7 +256,7 @@ template struct RegisterAnalysis : public RegisterPassBase { RegisterAnalysis(const char *PassArg, const char *Name, bool CFGOnly = false) - : RegisterPassBase(Name, PassArg, typeid(PassName), PassInfo::Analysis, + : RegisterPassBase(Name, PassArg, typeid(PassName), callDefaultCtor) { if (CFGOnly) setOnlyUsesCFG(); } diff --git a/llvm/lib/VMCore/Pass.cpp b/llvm/lib/VMCore/Pass.cpp index dc49ac6aeeca..a06f08464d34 100644 --- a/llvm/lib/VMCore/Pass.cpp +++ b/llvm/lib/VMCore/Pass.cpp @@ -136,7 +136,7 @@ void PMDebug::PrintArgumentInformation(const Pass *P) { } else { // Normal pass. Print argument information... // Print out arguments for registered passes that are _optimizations_ if (const PassInfo *PI = P->getPassInfo()) - if (PI->getPassType() & PassInfo::Optimization) + if (!PI->isAnalysisGroup()) std::cerr << " -" << PI->getPassArgument(); } } @@ -376,7 +376,7 @@ static std::map *AnalysisGroupInfoMap = 0; // RegisterAGBase::RegisterAGBase(const std::type_info &Interface, const std::type_info *Pass, bool isDefault) - : RegisterPassBase(Interface, PassInfo::AnalysisGroup), + : RegisterPassBase(Interface), ImplementationInfo(0), isDefaultImplementation(isDefault) { InterfaceInfo = const_cast(Pass::lookupPassInfo(Interface)); @@ -385,7 +385,7 @@ RegisterAGBase::RegisterAGBase(const std::type_info &Interface, registerPass(); InterfaceInfo = &PIObj; } - assert(InterfaceInfo->getPassType() == PassInfo::AnalysisGroup && + assert(PIObj.isAnalysisGroup() && "Trying to join an analysis group that is a normal pass!"); if (Pass) {