-
Notifications
You must be signed in to change notification settings - Fork 14.8k
[MLIR] Add tensor compiler dialect maintainers #150945
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@llvm/pr-subscribers-mlir Author: Renato Golin (rengolin) ChangesAs per https://discourse.llvm.org/t/mlir-project-maintainers/87189 @llvm/mlir-area-team Full diff: https://github.com/llvm/llvm-project/pull/150945.diff 1 Files Affected:
diff --git a/mlir/Maintainers.md b/mlir/Maintainers.md
index 7c852ef4e1771..4ea750eac6e50 100644
--- a/mlir/Maintainers.md
+++ b/mlir/Maintainers.md
@@ -61,3 +61,24 @@ particular for machine learning compilers.
[email protected] (email),
[@banach-space](https://github.com/banach-space) (GitHub),
banach-space (Discourse)
+
+### Dialects
+
+The `tensor` maintainer refers to the people working in the tensor compiler category, with the point-of-contact being the maintainers above.
+These are key MLIR dialects that will never become _unmaintained_.
+Named maintainers, if available, should be contacted first, as they're more active in those areas.
+
+#### Linear Algebra Dialects
+* ‘linalg’ Dialect (tensor)
+* Tensor Operator Set Architecture (TOSA) Dialect (sjarus)
+
+#### Type Dialects
+* ‘tensor’ Dialect (tensor)
+* ‘memref’ Dialect (tensor)
+* ‘vector’ Dialect (tensor + dcaballe, groverkss)
+* ‘sparse_tensor’ Dialect (aartbik, matthias-springer)
+
+#### Accessory Dialects
+* ‘bufferization’ Dialect (tensor, matthias-springer)
+* ‘ml_program’ Dialect (jpienaar)
+* ‘quant’ Dialect (unmaintained)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this be extended along the lines above, i.e. include the email (extract from commits) and github/discourse handle?
@sjarus @jpienaar @dcaballe @Groverkss @aartbik @matthias-springer please approve to indicate you agree to take up the maintainer role. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
|
||
The `tensor` maintainer refers to the people working in the tensor compiler category, with the point-of-contact being the maintainers above. | ||
These are key MLIR dialects that will never become _unmaintained_. | ||
Named maintainers, if available, should be contacted first, as they're more active in those areas. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/they're/they are/ ?
) Ref: https://discourse.llvm.org/t/mlir-project-maintainers/87189 See also: * #151721 * #150945 Compared to the original proposal, one change is included: * The `ub` dialect has @Hardcode84 as maintainer. Please accept to validate your nomination, let's keep new nominations for follow up PRs.
As per https://discourse.llvm.org/t/mlir-project-maintainers/87189
@llvm/mlir-area-team
@jpienaar
@banach-space