Skip to content

[DIALECT] Remove remaining nested tensor annotations in tensordesc and add a verifier#9984

Merged
ThomasRaoux merged 4 commits into
mainfrom
keren/fix-mlir
Apr 9, 2026
Merged

[DIALECT] Remove remaining nested tensor annotations in tensordesc and add a verifier#9984
ThomasRaoux merged 4 commits into
mainfrom
keren/fix-mlir

Conversation

@Jokeren
Copy link
Copy Markdown
Contributor

@Jokeren Jokeren commented Apr 9, 2026

No description provided.

@Jokeren Jokeren requested a review from ptillet as a code owner April 9, 2026 19:09
Copy link
Copy Markdown
Collaborator

@ThomasRaoux ThomasRaoux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how come those didn't fail lit tests in main?

@Jokeren
Copy link
Copy Markdown
Contributor Author

Jokeren commented Apr 9, 2026

how come those didn't fail lit tests in main?

Maybe there were merge conflicts

@ThomasRaoux
Copy link
Copy Markdown
Collaborator

how come those didn't fail lit tests in main?

Maybe there were merge conflicts

I just checked that it is passing. Maybe we should verify tensordesc type?

@Jokeren
Copy link
Copy Markdown
Contributor Author

Jokeren commented Apr 9, 2026

how come those didn't fail lit tests in main?

Maybe there were merge conflicts

I just checked that it is passing. Maybe we should verify tensordesc type?

Agree. working on it

@Jokeren Jokeren changed the title [DIALECT] Remove remaining nested tensor annotations in tensordesc [DIALECT] Remove remaining nested tensor annotations in tensordesc and add a verifier Apr 9, 2026
@ThomasRaoux ThomasRaoux merged commit 58bc676 into main Apr 9, 2026
8 of 9 checks passed
@ThomasRaoux ThomasRaoux deleted the keren/fix-mlir branch April 9, 2026 22:59
plognjen pushed a commit to plognjen/triton that referenced this pull request Apr 14, 2026
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.

2 participants