Fix A5 tmov treshape valid shape#687
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces a mechanism to correctly handle dynamic valid shapes during the normalization of TMOV operations for the A5 architecture. It implements a new internal attribute, __pto.a5_tmov_normalize_treshape, to track TReshapeOp instances and ensures that dynamic valid dimensions are properly materialized or swapped during layout transitions. These changes are integrated into the PTOA5NormalizeTMovPass, PTOMaterializeTileHandles, and PTOViewToMemref passes, supported by new test cases. I have no feedback to provide.
353c46f to
9569f9b
Compare
Codex Review该评论由 review 机器人自动更新。
SummaryPR #687 新增了 Findings
这里把 |
45618d5 to
875eac4
Compare
875eac4 to
2971d21
Compare
|
/run all |
|
已接收
页面会自动刷新,可以直接看当前阶段、排队情况和最近结果。 |
A3 板测完成(有跳过)
|
A5 板测成功
|
A3 板测失败
失败用例
|
A3 板测失败详情:PR #687rope_kv_cache
down_proj_residual
out_proj_residual
qwen3_decode_incore_5
|
Summary
pto.get_validshapeto read runtime tile valid_row/valid_col metadata, with EmitC lowering totile.GetValidRow()/tile.GetValidCol()get_validshape, swapping col/row without compile-time valid-dim lookupTesting
ninja -C build-wsl-pr567-verify tools/ptoas/ptoaspython3 /usr/lib/llvm-18/build/utils/lit/lit.py -a build-wsl-pr567-verify/test/lit/pto/get_validshape_emitc.pto build-wsl-pr567-verify/test/lit/pto/issue686_a5_tmov_treshape_dynamic_valid_shape.pto build-wsl-pr567-verify/test/lit/pto/issue686_a5_tmov_treshape_square_dynamic_valid_shape.pto build-wsl-pr567-verify/test/lit/pto/treshape_explicit_dynamic_valid_shape_preserved.pto build-wsl-pr567-verify/test/lit/pto/treshape_static_valid_shape_emitc.ptoptoas/FileCheckforissue660_trowexpandmul_set_validshape_preserves_alloc_valid.ptoptoas/FileCheckfortpush_tpop_dynamic_validshape_default_a5.pto --check-prefix=A5git diff --check