Initialize Pass with PassID.

The passID is not currently stored in Pass but this avoids the unused variable warning. The passID is used to uniquely identify passes, currently this is only stored/used in PassInfo.

PiperOrigin-RevId: 220485662

14 files changed, 21 insertions, 9 deletions

diff --git a/mlir/lib/Analysis/MemRefBoundCheck.cpp b/mlir/lib/Analysis/MemRefBoundCheck.cpp
index a7f0ebf4936..eb4ce56d429 100644
--- a/mlir/lib/Analysis/MemRefBoundCheck.cpp
+++ b/mlir/lib/Analysis/MemRefBoundCheck.cpp
@@ -38,7 +38,7 @@ namespace {
 
 /// Checks for out of bound memef access subscripts..
 struct MemRefBoundCheck : public FunctionPass, StmtWalker<MemRefBoundCheck> {
-  explicit MemRefBoundCheck() {}
+  explicit MemRefBoundCheck() : FunctionPass(&MemRefBoundCheck::passID) {}
 
   PassResult runOnMLFunction(MLFunction *f) override;
   // Not applicable to CFG functions.
diff --git a/mlir/lib/Analysis/MemRefDependenceCheck.cpp b/mlir/lib/Analysis/MemRefDependenceCheck.cpp
index 7a620c1a3a8..28a80762b94 100644
--- a/mlir/lib/Analysis/MemRefDependenceCheck.cpp
+++ b/mlir/lib/Analysis/MemRefDependenceCheck.cpp
@@ -40,7 +40,8 @@ namespace {
 struct MemRefDependenceCheck : public FunctionPass,
                                StmtWalker<MemRefDependenceCheck> {
   SmallVector<OperationStmt *, 4> loadsAndStores;
-  explicit MemRefDependenceCheck() {}
+  explicit MemRefDependenceCheck()
+      : FunctionPass(&MemRefDependenceCheck::passID) {}
 
   PassResult runOnMLFunction(MLFunction *f) override;
   // Not applicable to CFG functions.
diff --git a/mlir/lib/Transforms/CFGFunctionViewGraph.cpp b/mlir/lib/Transforms/CFGFunctionViewGraph.cpp
index 810264cb35e..d29708d4fef 100644
--- a/mlir/lib/Transforms/CFGFunctionViewGraph.cpp
+++ b/mlir/lib/Transforms/CFGFunctionViewGraph.cpp
@@ -76,7 +76,8 @@ namespace {
 struct PrintCFGPass : public FunctionPass {
   PrintCFGPass(llvm::raw_ostream &os = llvm::errs(), bool shortNames = false,
                const llvm::Twine &title = "")
-      : os(os), shortNames(shortNames), title(title) {}
+      : FunctionPass(&PrintCFGPass::passID), os(os), shortNames(shortNames),
+        title(title) {}
   PassResult runOnCFGFunction(CFGFunction *function) override {
     mlir::writeGraph(os, function, shortNames, title);
     return success();
diff --git a/mlir/lib/Transforms/Canonicalizer.cpp b/mlir/lib/Transforms/Canonicalizer.cpp
index 3a62f132459..e6f3a2b0550 100644
--- a/mlir/lib/Transforms/Canonicalizer.cpp
+++ b/mlir/lib/Transforms/Canonicalizer.cpp
@@ -34,6 +34,7 @@ namespace {
 
 /// Canonicalize operations in functions.
 struct Canonicalizer : public FunctionPass {
+  Canonicalizer() : FunctionPass(&Canonicalizer::passID) {}
   PassResult runOnFunction(Function *fn) override;
 
   static char passID;
diff --git a/mlir/lib/Transforms/ComposeAffineMaps.cpp b/mlir/lib/Transforms/ComposeAffineMaps.cpp
index 61e2e8f2e83..84507b91703 100644
--- a/mlir/lib/Transforms/ComposeAffineMaps.cpp
+++ b/mlir/lib/Transforms/ComposeAffineMaps.cpp
@@ -44,7 +44,7 @@ namespace {
 struct ComposeAffineMaps : public FunctionPass, StmtWalker<ComposeAffineMaps> {
   std::vector<OperationStmt *> affineApplyOpsToErase;
 
-  explicit ComposeAffineMaps() {}
+  explicit ComposeAffineMaps() : FunctionPass(&ComposeAffineMaps::passID) {}
   using StmtListType = llvm::iplist<Statement>;
   void walk(StmtListType::iterator Start, StmtListType::iterator End);
   void visitOperationStmt(OperationStmt *stmt);
diff --git a/mlir/lib/Transforms/ConstantFold.cpp b/mlir/lib/Transforms/ConstantFold.cpp
index 9005c2bbf48..15a5db15d73 100644
--- a/mlir/lib/Transforms/ConstantFold.cpp
+++ b/mlir/lib/Transforms/ConstantFold.cpp
@@ -27,6 +27,8 @@ using namespace mlir;
 namespace {
 /// Simple constant folding pass.
 struct ConstantFold : public FunctionPass, StmtWalker<ConstantFold> {
+  ConstantFold() : FunctionPass(&ConstantFold::passID) {}
+
   // All constants in the function post folding.
   SmallVector<SSAValue *, 8> existingConstants;
   // Operation statements that were folded and that need to be erased.
diff --git a/mlir/lib/Transforms/ConvertToCFG.cpp b/mlir/lib/Transforms/ConvertToCFG.cpp
index b36717d272f..8ffa23ca102 100644
--- a/mlir/lib/Transforms/ConvertToCFG.cpp
+++ b/mlir/lib/Transforms/ConvertToCFG.cpp
@@ -66,7 +66,7 @@ namespace {
 // ModuleConverter class does CFG conversion for the whole module.
 class ModuleConverter : public ModulePass {
 public:
-  explicit ModuleConverter() {}
+  explicit ModuleConverter() : ModulePass(&ModuleConverter::passID) {}
 
   PassResult runOnModule(Module *m) override;
 
diff --git a/mlir/lib/Transforms/LoopFusion.cpp b/mlir/lib/Transforms/LoopFusion.cpp
index ae4647e143d..87657aeb359 100644
--- a/mlir/lib/Transforms/LoopFusion.cpp
+++ b/mlir/lib/Transforms/LoopFusion.cpp
@@ -42,7 +42,7 @@ namespace {
 // TODO(andydavis) Extend this pass to check for fusion preventing dependences,
 // and add support for more general loop fusion algorithms.
 struct LoopFusion : public FunctionPass {
-  LoopFusion() {}
+  LoopFusion() : FunctionPass(&LoopFusion::passID) {}
 
   PassResult runOnMLFunction(MLFunction *f) override;
   static char passID;
diff --git a/mlir/lib/Transforms/LoopTiling.cpp b/mlir/lib/Transforms/LoopTiling.cpp
index 3bff008942c..8efadfc44bb 100644
--- a/mlir/lib/Transforms/LoopTiling.cpp
+++ b/mlir/lib/Transforms/LoopTiling.cpp
@@ -40,6 +40,7 @@ namespace {
 
 /// A pass to perform loop tiling on all suitable loop nests of an MLFunction.
 struct LoopTiling : public FunctionPass {
+  LoopTiling() : FunctionPass(&LoopTiling::passID) {}
   PassResult runOnMLFunction(MLFunction *f) override;
   constexpr static unsigned kDefaultTileSize = 32;
 
diff --git a/mlir/lib/Transforms/LoopUnroll.cpp b/mlir/lib/Transforms/LoopUnroll.cpp
index ae09098f9d5..76e484c6a10 100644
--- a/mlir/lib/Transforms/LoopUnroll.cpp
+++ b/mlir/lib/Transforms/LoopUnroll.cpp
@@ -58,7 +58,8 @@ struct LoopUnroll : public FunctionPass {
 
   explicit LoopUnroll(Optional<unsigned> unrollFactor = None,
                       Optional<bool> unrollFull = None)
-      : unrollFactor(unrollFactor), unrollFull(unrollFull) {}
+      : FunctionPass(&LoopUnroll::passID), unrollFactor(unrollFactor),
+        unrollFull(unrollFull) {}
 
   PassResult runOnMLFunction(MLFunction *f) override;
 
diff --git a/mlir/lib/Transforms/LoopUnrollAndJam.cpp b/mlir/lib/Transforms/LoopUnrollAndJam.cpp
index ce6e939fae8..45ca9dd98df 100644
--- a/mlir/lib/Transforms/LoopUnrollAndJam.cpp
+++ b/mlir/lib/Transforms/LoopUnrollAndJam.cpp
@@ -71,7 +71,8 @@ struct LoopUnrollAndJam : public FunctionPass {
   static const unsigned kDefaultUnrollJamFactor = 4;
 
   explicit LoopUnrollAndJam(Optional<unsigned> unrollJamFactor = None)
-      : unrollJamFactor(unrollJamFactor) {}
+      : FunctionPass(&LoopUnrollAndJam::passID),
+        unrollJamFactor(unrollJamFactor) {}
 
   PassResult runOnMLFunction(MLFunction *f) override;
   bool runOnForStmt(ForStmt *forStmt);
diff --git a/mlir/lib/Transforms/PipelineDataTransfer.cpp b/mlir/lib/Transforms/PipelineDataTransfer.cpp
index 52052e09d7b..c3f131f407f 100644
--- a/mlir/lib/Transforms/PipelineDataTransfer.cpp
+++ b/mlir/lib/Transforms/PipelineDataTransfer.cpp
@@ -41,6 +41,7 @@ namespace {
 
 struct PipelineDataTransfer : public FunctionPass,
                               StmtWalker<PipelineDataTransfer> {
+  PipelineDataTransfer() : FunctionPass(&PipelineDataTransfer::passID) {}
   PassResult runOnMLFunction(MLFunction *f) override;
   PassResult runOnForStmt(ForStmt *forStmt);
 
diff --git a/mlir/lib/Transforms/SimplifyAffineExpr.cpp b/mlir/lib/Transforms/SimplifyAffineExpr.cpp
index 92d585f31bc..06f3f8f44e2 100644
--- a/mlir/lib/Transforms/SimplifyAffineExpr.cpp
+++ b/mlir/lib/Transforms/SimplifyAffineExpr.cpp
@@ -38,7 +38,8 @@ namespace {
 //  ML functions and CFG functions.
 struct SimplifyAffineStructures : public FunctionPass,
                                   StmtWalker<SimplifyAffineStructures> {
-  explicit SimplifyAffineStructures() {}
+  explicit SimplifyAffineStructures()
+      : FunctionPass(&SimplifyAffineStructures::passID) {}
 
   PassResult runOnMLFunction(MLFunction *f) override;
   // Does nothing on CFG functions for now. No reusable walkers/visitors exist
diff --git a/mlir/lib/Transforms/Vectorize.cpp b/mlir/lib/Transforms/Vectorize.cpp
index 63969af451f..90ee87563de 100644
--- a/mlir/lib/Transforms/Vectorize.cpp
+++ b/mlir/lib/Transforms/Vectorize.cpp
@@ -195,6 +195,8 @@ static std::vector<MLFunctionMatcher> makePatterns() {
 namespace {
 
 struct Vectorize : public FunctionPass {
+  Vectorize() : FunctionPass(&Vectorize::passID) {}
+
   PassResult runOnMLFunction(MLFunction *f) override;
 
   // Thread-safe RAII contexts local to pass, BumpPtrAllocator freed on exit.