Skip to content

TVM/MLIR: refact with PlainNodeSchedule/LoopNest - #110

Merged
guillon merged 2 commits into
xtc-tools:mainfrom
guillon:dev/cguillon/tvm-refactor
Aug 24, 2026
Merged

TVM/MLIR: refact with PlainNodeSchedule/LoopNest#110
guillon merged 2 commits into
xtc-tools:mainfrom
guillon:dev/cguillon/tvm-refactor

Conversation

@guillon

@guillon guillon commented Jul 28, 2026

Copy link
Copy Markdown
Member

Motivation

Depends upon PR #106 . Ref only to last 2 commit.

Refactor TVM Schedulers TE/TIR to use a new PlainNodeScheduler/PlainNodeSchedule objects and convert through LoopNestBuilder before schedule is emitted.

Refactor MlirNodeScheduler to use PlainNodeScheduler/PlainNodeSchedule. Still use MlirNodeSchedule (actually derived from PlainNodeSchedule) for Transform dialect generation. Though refactored loop_nest construction with LoopNestBuilder on call to get_loop_nest().

Description

Add PlainNodeSchedule[r] and conversion to LoopNest through LoopNestBuilder.

Update TVM TE Emitter and TIR Emmiter to use LoopNest instead of the former TVMPlainSchedule object.

Update MlirNodeSchedule to use these also.

Commits

Ref to last 2 commit:

  • tvm: refactor for TVM
  • mlir: refactor for MLIR

Discussion

n/a

@codecov

codecov Bot commented Jul 28, 2026

Copy link
Copy Markdown

@guillon
guillon force-pushed the dev/cguillon/tvm-refactor branch 2 times, most recently from 6bebe42 to ffb6ee2 Compare July 28, 2026 13:17
@guillon guillon changed the title TVM: refactor TVMScheduler with PlainNodeSchedule/LoopNest TVM/MIR: refact with PlainNodeSchedule/LoopNest Jul 28, 2026
@guillon
guillon marked this pull request as ready for review July 28, 2026 14:23
@guillon
guillon requested a review from qaco July 28, 2026 14:23
@guillon guillon changed the title TVM/MIR: refact with PlainNodeSchedule/LoopNest TVM/MLIR: refact with PlainNodeSchedule/LoopNest Jul 28, 2026
@guillon guillon added the refactor Refactoring or simplification of code label Jul 28, 2026
@guillon
guillon force-pushed the dev/cguillon/tvm-refactor branch from 3b361a1 to c93629e Compare July 28, 2026 15:55
Comment thread src/xtc/backends/mlir/MlirNodeScheduler.py Outdated

@qaco qaco left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great work and idea! Just a dead code remark above

@guillon

guillon commented Aug 24, 2026

Copy link
Copy Markdown
Member Author

@qaco, can you check or delegate PR #106 and PR #113 ?

@guillon
guillon force-pushed the dev/cguillon/tvm-refactor branch from c93629e to 4788cc7 Compare August 24, 2026 10:15
@guillon
guillon force-pushed the dev/cguillon/tvm-refactor branch 2 times, most recently from 4788cc7 to 84c8486 Compare August 24, 2026 10:27
@guillon
guillon merged commit 21571c4 into xtc-tools:main Aug 24, 2026
4 checks passed
@guillon
guillon deleted the dev/cguillon/tvm-refactor branch August 24, 2026 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Refactoring or simplification of code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants