Skip to content

feat(fusion): port tile fusion onto feature-vpto-backend#368

Open
Zhendong404 wants to merge 1 commit into
mouliangyu:feature-vpto-backendfrom
Zhendong404:port-tile-fusion
Open

feat(fusion): port tile fusion onto feature-vpto-backend#368
Zhendong404 wants to merge 1 commit into
mouliangyu:feature-vpto-backendfrom
Zhendong404:port-tile-fusion

Conversation

@Zhendong404
Copy link
Copy Markdown
Collaborator

Summary

  • reintroduce VPTO tile fusion pipeline pieces on top of feature-vpto-backend
  • add tile fusion analyses / passes / verifier coverage and lit regressions
  • include the paged-attention fusion regression and the PTOViewToMemref fix for fusion-region yielded results

Validation

  • llvm-lit -sv test/lit/tile_fusion/op_fusion_paged_attention_online_update.pto
  • ptoas --pto-backend=vpto --pto-arch=a5 --pto-level=level2 --enable-op-fusion --emit-pto-ir test/lit/tile_fusion/op_fusion_paged_attention_online_update.pto -o /dev/null

Notes

  • the copied paged-attention regression is currently kept as a fusion/container regression; full emit-vpto on that sample still hits downstream TileLang template coverage gaps unrelated to the fusion-region result fix.

@Zhendong404 Zhendong404 marked this pull request as ready for review May 14, 2026 13:23
Comment thread tools/ptoas/ptoas.cpp
static void lowerPTOToVPTOBackend(PassManager &pm, ModuleOp module, int argc,
char **argv) {
auto &kernelModulePM = pm.nest<ModuleOp>();
auto moduleArchAttr =
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

不建议在构建 pm 的中途做 module 的判断,建议放在 pm 初始化时。

Comment thread tools/ptoas/ptoas.cpp
return 1;
}

if (arch != "a5" && enableOpFusion) {
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

看起来像是一些孤立的报错,建议在实际的边界处报错

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