[Mlir-commits] [mlir] af1ca08 - [mlir][EmitC]Remove the Unused Option in the `wrap-emitc-func-in-class` Pass (#150577)
llvmlistbot at llvm.org
llvmlistbot at llvm.org
Fri Jul 25 09:30:59 PDT 2025
Author: Jaden Angella
Date: 2025-07-25T09:30:55-07:00
New Revision: af1ca08ddbafc729a44ba3833268c344c835ca8e
URL: https://github.com/llvm/llvm-project/commit/af1ca08ddbafc729a44ba3833268c344c835ca8e
DIFF: https://github.com/llvm/llvm-project/commit/af1ca08ddbafc729a44ba3833268c344c835ca8e.diff
LOG: [mlir][EmitC]Remove the Unused Option in the `wrap-emitc-func-in-class` Pass (#150577)
This patch removes the option `namedAttribute` which is not used in this
pass.
Added:
Modified:
mlir/include/mlir/Dialect/EmitC/Transforms/Passes.td
mlir/include/mlir/Dialect/EmitC/Transforms/Transforms.h
mlir/lib/Dialect/EmitC/Transforms/WrapFuncInClass.cpp
mlir/test/Dialect/EmitC/wrap_emitc_func_in_class.mlir
Removed:
################################################################################
diff --git a/mlir/include/mlir/Dialect/EmitC/Transforms/Passes.td b/mlir/include/mlir/Dialect/EmitC/Transforms/Passes.td
index 74c49132b61f6..1893c101e735b 100644
--- a/mlir/include/mlir/Dialect/EmitC/Transforms/Passes.td
+++ b/mlir/include/mlir/Dialect/EmitC/Transforms/Passes.td
@@ -51,11 +51,6 @@ def WrapFuncInClassPass : Pass<"wrap-emitc-func-in-class"> {
```
}];
let dependentDialects = ["emitc::EmitCDialect"];
- let options = [Option<
- "namedAttribute", "named-attribute", "std::string",
- /*default=*/"",
- "Attribute key used to extract field names from function argument's "
- "dictionary attributes">];
}
#endif // MLIR_DIALECT_EMITC_TRANSFORMS_PASSES
diff --git a/mlir/include/mlir/Dialect/EmitC/Transforms/Transforms.h b/mlir/include/mlir/Dialect/EmitC/Transforms/Transforms.h
index a4e8fe10ff853..bdf6d0985e6db 100644
--- a/mlir/include/mlir/Dialect/EmitC/Transforms/Transforms.h
+++ b/mlir/include/mlir/Dialect/EmitC/Transforms/Transforms.h
@@ -29,8 +29,7 @@ ExpressionOp createExpression(Operation *op, OpBuilder &builder);
void populateExpressionPatterns(RewritePatternSet &patterns);
/// Populates 'patterns' with func-related patterns.
-void populateFuncPatterns(RewritePatternSet &patterns,
- StringRef namedAttribute);
+void populateFuncPatterns(RewritePatternSet &patterns);
} // namespace emitc
} // namespace mlir
diff --git a/mlir/lib/Dialect/EmitC/Transforms/WrapFuncInClass.cpp b/mlir/lib/Dialect/EmitC/Transforms/WrapFuncInClass.cpp
index 612e8099eaf35..fa05ad8063b99 100644
--- a/mlir/lib/Dialect/EmitC/Transforms/WrapFuncInClass.cpp
+++ b/mlir/lib/Dialect/EmitC/Transforms/WrapFuncInClass.cpp
@@ -31,7 +31,7 @@ struct WrapFuncInClassPass
Operation *rootOp = getOperation();
RewritePatternSet patterns(&getContext());
- populateFuncPatterns(patterns, namedAttribute);
+ populateFuncPatterns(patterns);
walkAndApplyPatterns(rootOp, std::move(patterns));
}
@@ -43,8 +43,8 @@ struct WrapFuncInClassPass
class WrapFuncInClass : public OpRewritePattern<emitc::FuncOp> {
public:
- WrapFuncInClass(MLIRContext *context, StringRef attrName)
- : OpRewritePattern<emitc::FuncOp>(context), attributeName(attrName) {}
+ WrapFuncInClass(MLIRContext *context)
+ : OpRewritePattern<emitc::FuncOp>(context) {}
LogicalResult matchAndRewrite(emitc::FuncOp funcOp,
PatternRewriter &rewriter) const override {
@@ -101,12 +101,8 @@ class WrapFuncInClass : public OpRewritePattern<emitc::FuncOp> {
rewriter.replaceOp(funcOp, newClassOp);
return success();
}
-
-private:
- StringRef attributeName;
};
-void mlir::emitc::populateFuncPatterns(RewritePatternSet &patterns,
- StringRef namedAttribute) {
- patterns.add<WrapFuncInClass>(patterns.getContext(), namedAttribute);
+void mlir::emitc::populateFuncPatterns(RewritePatternSet &patterns) {
+ patterns.add<WrapFuncInClass>(patterns.getContext());
}
diff --git a/mlir/test/Dialect/EmitC/wrap_emitc_func_in_class.mlir b/mlir/test/Dialect/EmitC/wrap_emitc_func_in_class.mlir
index c67a0c197fcd9..029fa78a3f528 100644
--- a/mlir/test/Dialect/EmitC/wrap_emitc_func_in_class.mlir
+++ b/mlir/test/Dialect/EmitC/wrap_emitc_func_in_class.mlir
@@ -1,4 +1,4 @@
-// RUN: mlir-opt --wrap-emitc-func-in-class='named-attribute=emitc.name_hint' %s | FileCheck %s
+// RUN: mlir-opt --wrap-emitc-func-in-class %s | FileCheck %s
module attributes { } {
emitc.func @model(%arg0: !emitc.array<1xf32> {emitc.name_hint = "another_feature"},
More information about the Mlir-commits
mailing list