Skip to content

Update mlir-air/aie and port annotation-based transform scripts#13

Merged
erwei-xilinx merged 1 commit into
mainfrom
update-mlir-air-and-transform-scripts
Mar 5, 2026
Merged

Update mlir-air/aie and port annotation-based transform scripts#13
erwei-xilinx merged 1 commit into
mainfrom
update-mlir-air-and-transform-scripts

Conversation

@erwei-xilinx

@erwei-xilinx erwei-xilinx commented Mar 5, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Update mlir-air to 4ef22a2 (PRs #1344, #1347, #1349, #1352, #1358) and mlir-aie to c5d4befd
  • Replace fragile positional split_handle patterns with annotation-based matching in matmul scripts
  • Use consolidated 2-arg hoist_loop_invariant_transfers with auto-discovery of loop-invariant transfer pairs
  • Port softmax/layernorm data-flow navigation from linalg.reduce anchors
  • Update driver.py fallback script from PDL to named_sequence / !transform.any_op

Test plan

  • matmul PASS on Strix AIE2P hardware
  • test_softmax PASS on Strix AIE2P hardware
  • test_layernorm PASS on Strix AIE2P hardware
  • vec-add PASS on Strix AIE2P hardware
  • CI build validation (ubuntu-latest)
  • Phoenix AIE2 hardware validation (if available)

🤖 Generated with Claude Code

Update mlir-air to 4ef22a2 and mlir-aie to c5d4befd to pick up transform
dialect improvements from PRs #1344, #1347, #1349, #1352, #1358:

- Replace fragile split_handle patterns with annotation-based matching in
  matmul scripts (copy_a_loop, copy_b_loop, k_reduction_loop, compute_forall,
  matmul_compute, init_fill, prologue/epilogue_forall/herd annotations)
- Use consolidated 2-arg hoist_loop_invariant_transfers (scope_op, loop_op)
  with auto-discovery, eliminating 8-way read + 4-way write split_handle
- Port softmax data-flow navigation from linalg.reduce anchors (PR #1358)
- Port layernorm data-flow navigation from linalg.reduce anchors (PR #1358)
- Update driver.py fallback script from PDL wrapper to named_sequence
- All scripts use transform.readonly entry point

Tested: 4/4 examples pass on Strix AIE2P hardware (matmul, softmax,
layernorm, vec-add). split_handle count reduced from 39 to 33 (remaining
are stable post-fusion patterns matching mlir-air reference scripts).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@erwei-xilinx erwei-xilinx merged commit 548c4f3 into main Mar 5, 2026
8 of 9 checks passed
@erwei-xilinx erwei-xilinx deleted the update-mlir-air-and-transform-scripts branch March 5, 2026 03:42
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.

1 participant