Skip to content

Conversation

@silee2
Copy link
Contributor

@silee2 silee2 commented Nov 7, 2025

XeGPU and XeVM dialect has assigned maintainers, but related folders currently lack code owners.
Add charithaintc and Jianhui-Li as code owner for XeGPU related folders.
Add silee2 as code owner for XeVM related folders.
Note:
charithaintc is current maintainer of XeGPU dialect.
silee2 is current maintainer of XeVM dialect.

for XeGPU and XeVM dialect related folders.
And add Jianhui-Li for XeGPU related folders.
@silee2 silee2 changed the title [MLIR][CODEOWNER] Add XeGPU and XeVM codeowners [MLIR][CODEOWNERS] Add XeGPU and XeVM codeowners Nov 7, 2025
@silee2 silee2 requested a review from rengolin November 12, 2025 22:03
@silee2
Copy link
Contributor Author

silee2 commented Nov 12, 2025

@rengolin Who should I ask for review? Surprised to see no code owner assigned for CODEOWNERS file.

@silee2
Copy link
Contributor Author

silee2 commented Nov 12, 2025

@tkarna I cannot add you as a reviewer. Do you lack write permit?

@tkarna
Copy link
Contributor

tkarna commented Nov 13, 2025

@tkarna I cannot add you as a reviewer. Do you lack write permit?

That's right, my commit access request is still pending. This looks good to me, but I'm not sure if I can be marked as a codeowner before I have permissions.

@rengolin
Copy link
Member

@rengolin Who should I ask for review? Surprised to see no code owner assigned for CODEOWNERS file.

This is mostly self-assigned. It only means "I want to receive notifications for PRs and issues on those areas".

I have approved.

@silee2
Copy link
Contributor Author

silee2 commented Nov 13, 2025

@tkarna I cannot add you as a reviewer. Do you lack write permit?

That's right, my commit access request is still pending. This looks good to me, but I'm not sure if I can be marked as a codeowner before I have permissions.

I'll remove for now and can add you later once you get write permit.

@silee2 silee2 merged commit a3cdef8 into llvm:main Nov 13, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants