Skip to content

Fix construction of LoopNest with Split. - #113

Open
guillon wants to merge 10 commits into
xtc-tools:mainfrom
guillon:dev/cguillon/fix-split
Open

Fix construction of LoopNest with Split.#113
guillon wants to merge 10 commits into
xtc-tools:mainfrom
guillon:dev/cguillon/fix-split

Conversation

@guillon

@guillon guillon commented Aug 11, 2026

Copy link
Copy Markdown
Member

Motivation

Depends upon PR #110. Refer to last commit only

Fix support for nested splits when generating plain schedule and LoopNest from descript and low level API.

Make LoopNest and other dataclass frozen (at least at first level) in order to avoid abuse.

Description

Add recursive construction of LoopNest.

Update PlainNodeSchedule to fully qualify root path.

Commits

Ref to last commit

Discussion

For now the Mlir transform dialect generation does not seem to support well nested splits.

For now the TVM scheduler does not implement split.

Both will be fixed in some further PRs. There the objective was to fix schedules/loopnest constructions first.

@guillon guillon self-assigned this Aug 11, 2026
@guillon
guillon force-pushed the dev/cguillon/fix-split branch from 75661b8 to fd57fb5 Compare August 11, 2026 15:37
@guillon
guillon force-pushed the dev/cguillon/fix-split branch from fd57fb5 to 7888d34 Compare August 11, 2026 15:59
@guillon
guillon marked this pull request as ready for review August 11, 2026 18:31
@guillon
guillon requested a review from qaco August 11, 2026 18:31
@guillon guillon assigned qaco and unassigned guillon Aug 11, 2026
@guillon guillon added the bug Something isn't working label Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants