[PATCH] D74068: [MLIR] Rename MemRefBoundCheck.cpp -> TestMemRefBoundCheck.cpp r=rriddle

[Stephen Neuendorffer via Phabricator via llvm-commits]

stephenneuendorffer created this revision.
stephenneuendorffer added a reviewer: rriddle.
Herald added subscribers: llvm-commits, Joonsoo, liufengdb, aartbik, lucyrfox, mgester, arpith-jacob, nicolasvasilache, antiagainst, shauheen, burmako, jpienaar, mehdi_amini, mgorny.
Herald added a project: LLVM.

This makes it consistent with other test passes.

Depends on D74067 <https://reviews.llvm.org/D74067>


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D74068

Files:
  mlir/include/mlir/Analysis/Passes.h
  mlir/test/Transforms/memref-bound-check.mlir
  mlir/test/lib/Transforms/CMakeLists.txt
  mlir/test/lib/Transforms/MemRefBoundCheck.cpp
  mlir/test/lib/Transforms/TestMemRefBoundCheck.cpp


Index: mlir/test/lib/Transforms/TestMemRefBoundCheck.cpp
===================================================================
--- mlir/test/lib/Transforms/TestMemRefBoundCheck.cpp
+++ mlir/test/lib/Transforms/TestMemRefBoundCheck.cpp
@@ -29,17 +29,17 @@
 namespace {
 
 /// Checks for out of bound memef access subscripts..
-struct MemRefBoundCheck : public FunctionPass<MemRefBoundCheck> {
+struct TestMemRefBoundCheck : public FunctionPass<TestMemRefBoundCheck> {
   void runOnFunction() override;
 };
 
 } // end anonymous namespace
 
-std::unique_ptr<OpPassBase<FuncOp>> mlir::createMemRefBoundCheckPass() {
-  return std::make_unique<MemRefBoundCheck>();
+std::unique_ptr<OpPassBase<FuncOp>> mlir::createTestMemRefBoundCheckPass() {
+  return std::make_unique<TestMemRefBoundCheck>();
 }
 
-void MemRefBoundCheck::runOnFunction() {
+void TestMemRefBoundCheck::runOnFunction() {
   getFunction().walk([](Operation *opInst) {
     TypeSwitch<Operation *>(opInst).Case<AffineLoadOp, AffineStoreOp>(
         [](auto op) { boundCheckLoadOrStoreOp(op); });
@@ -48,6 +48,6 @@
   });
 }
 
-static PassRegistration<MemRefBoundCheck>
-    memRefBoundCheck("memref-bound-check",
+static PassRegistration<TestMemRefBoundCheck>
+    memRefBoundCheck("test-memref-bound-check",
                      "Check memref access bounds in a Function");
Index: mlir/test/lib/Transforms/CMakeLists.txt
===================================================================
--- mlir/test/lib/Transforms/CMakeLists.txt
+++ mlir/test/lib/Transforms/CMakeLists.txt
@@ -10,7 +10,7 @@
   TestLoopMapping.cpp
   TestLoopParametricTiling.cpp
   TestOpaqueLoc.cpp
-  MemRefBoundCheck.cpp
+  TestMemRefBoundCheck.cpp
   TestMemRefDependenceCheck.cpp
   TestMemRefStrideCalculation.cpp
   TestParallelismDetection.cpp
Index: mlir/test/Transforms/memref-bound-check.mlir
===================================================================
--- mlir/test/Transforms/memref-bound-check.mlir
+++ mlir/test/Transforms/memref-bound-check.mlir
@@ -1,4 +1,4 @@
-// RUN: mlir-opt %s -memref-bound-check -split-input-file -verify-diagnostics | FileCheck %s
+// RUN: mlir-opt %s -test-memref-bound-check -split-input-file -verify-diagnostics | FileCheck %s
 
 // -----
 
Index: mlir/include/mlir/Analysis/Passes.h
===================================================================
--- mlir/include/mlir/Analysis/Passes.h
+++ mlir/include/mlir/Analysis/Passes.h
@@ -23,7 +23,7 @@
 template <typename T> class OpPassBase;
 
 /// Creates a pass to check memref accesses in a Function.
-std::unique_ptr<OpPassBase<FuncOp>> createMemRefBoundCheckPass();
+std::unique_ptr<OpPassBase<FuncOp>> createTestMemRefBoundCheckPass();
 
 /// Creates a pass to check memref access dependences in a Function.
 std::unique_ptr<OpPassBase<FuncOp>> createTestMemRefDependenceCheckPass();


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D74068.242657.patch
Type: text/x-patch
Size: 2824 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200205/c540c1cb/attachment.bin>