Skip to content

Feature vpto backend tinsert new2【不合入】#470

Open
erhsh wants to merge 6 commits into
mouliangyu:feature-vpto-backendfrom
erhsh:feature-vpto-backend_tinsert_new2
Open

Feature vpto backend tinsert new2【不合入】#470
erhsh wants to merge 6 commits into
mouliangyu:feature-vpto-backendfrom
erhsh:feature-vpto-backend_tinsert_new2

Conversation

@erhsh

@erhsh erhsh commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

caojian5 added 5 commits June 2, 2026 20:23
1. Reorder ckernel params (constraints before priority) to match base
   branch, preventing duplicate 'constraints' param on merge.
2. Update tinsert_fp_a3_lowering.pto to expect TINSERT_FP< instead of
   TINSERT_FP( since the PR adds template params to the emission.
@erhsh erhsh force-pushed the feature-vpto-backend_tinsert_new2 branch from f7554f7 to 4d9aa19 Compare June 2, 2026 13:05
The VPTO emission pipeline failed on Acc->Vec NZ tinsert cases because
PTOMaterializeTileHandles wraps VEC memref operands with a new alloc_tile
that uses the DEFAULT tile config (no layout/fractal). When ExpandTileOp
then invokes the daemon, the daemon-generated template function expects
the original config (blayout=col_major, slayout=row_major, fractal=512),
causing bridgeOperandToType to insert an UnrealizedConversionCastOp
between the two tile_buf types.

After inlining, pto.tile_buf_addr saw the cast result instead of the
alloc_tile result, triggering the 'expected tile_buf to be defined by
the active materialized tile-handle bridge' error.

Fix resolveTileHandle to unwrap any chain of single-operand
UnrealizedConversionCastOp before matching AllocTileOp / MaterializeTileOp,
and add a cleanup pass to erase dead TileBuf-to-TileBuf casts that become
unused after folding.
@erhsh erhsh force-pushed the feature-vpto-backend_tinsert_new2 branch from 4d9aa19 to 1e51f7f Compare June 3, 2026 01:31
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