Skip to content

Update for latest triton-xdna#190

Merged
ypapadop-amd merged 1 commit into
hsa-backendfrom
ypapadop-amd/triton-xdna-update
May 19, 2026
Merged

Update for latest triton-xdna#190
ypapadop-amd merged 1 commit into
hsa-backendfrom
ypapadop-amd/triton-xdna-update

Conversation

@ypapadop-amd

Copy link
Copy Markdown
Owner

No description provided.

@ypapadop-amd ypapadop-amd self-assigned this May 19, 2026
Copilot AI review requested due to automatic review settings May 19, 2026 18:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates GGML’s HSA Triton (Triton-XDNA) integration to align with newer Triton-XDNA expectations by simplifying the vector-add transform script and adjusting how the optional transform script path is passed into config_context().

Changes:

  • Refactor vecadd_aie2.mlir to use transform.include calls to shared transform-library sequences instead of inlining all transform steps.
  • Update build_triton.py to pass dataclasses.MISSING as the default sentinel for transform_tiling_script (replacing the previous _UNSET usage).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/ggml-hsa/kernels/triton/vecadd_aie2.mlir Switches vecadd AIE2 transform pipeline to shared transform.include sequences for newer Triton-XDNA flows.
src/ggml-hsa/kernels/build_triton.py Updates config_context() invocation to use MISSING as the “not provided” default for transform_tiling_script.

Comment thread src/ggml-hsa/kernels/triton/vecadd_aie2.mlir
@ypapadop-amd ypapadop-amd merged commit 723bd26 into hsa-backend May 19, 2026
5 checks passed
@ypapadop-amd ypapadop-amd deleted the ypapadop-amd/triton-xdna-update branch May 19, 2026 18:35
ypapadop-amd added a commit that referenced this pull request Jun 26, 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