address comments

Author: Peiming Liu

mlir/include/mlir/Dialect/SparseTensor/Transforms/Passes.td

@@ -471,7 +471,7 @@ def SparsificationAndBufferization : Pass<"sparsification-and-bufferization", "M
 def SparseSpaceCollapse : Pass<"sparse-space-collapse", "func::FuncOp"> {
   let summary = "(experimental) sparse space collpasing pass";
   let description = [{
-     This pass collapse consecutive sparse spaces (extracted from the same tensor)
+     This pass collapses consecutive sparse spaces (extracted from the same tensor)
      into one multi-dimensional space. The pass is not yet stablized.
   }];
   let constructor = "mlir::createSparseSpaceCollapsePass()";

mlir/lib/Dialect/SparseTensor/Transforms/SparseSpaceCollapse.cpp

@@ -14,11 +14,14 @@
 #include "mlir/Dialect/SparseTensor/Transforms/Passes.h"
 
 namespace mlir {
-
 #define GEN_PASS_DEF_SPARSESPACECOLLAPSE
 #include "mlir/Dialect/SparseTensor/Transforms/Passes.h.inc"
+} // namespace mlir
+
+using namespace mlir;
+using namespace sparse_tensor;
 
-namespace sparse_tensor {
+namespace {
 
 struct CollapseSpaceInfo {
   ExtractIterSpaceOp space;
@@ -174,10 +177,8 @@ struct SparseSpaceCollapsePass
   }
 };
 
-} // namespace sparse_tensor
+} // namespace
 
-std::unique_ptr<Pass> createSparseSpaceCollapsePass() {
-  return std::make_unique<sparse_tensor::SparseSpaceCollapsePass>();
+std::unique_ptr<Pass> mlir::createSparseSpaceCollapsePass() {
+  return std::make_unique<SparseSpaceCollapsePass>();
 }
-
-} // namespace mlir