Skip to content

Conversation

@MasterJH5574
Copy link
Contributor

PR #14182 introduces the default GPU schedule pass, which makes uses
of the BlockCollector in Meta-Schedule. When requested, the
BlockCollector goes through every function in the input IRModule,
while in the case of applying default GPU schedule where we apply
schedule to one PrimFunc at a time, going through every PrimFunc is
wasteful.

Therefore, this PR enhances the BlockCollector so that it only focuses
on the function being worked on.

PR apache#11999 introduces the sugar method `work_on` to TIR Schedule, with
a field `func_working_on_` newly added to the ScheduleNode. In some
cases we may want to know which function a ScheduleNode is working on,
which is not supported previously.

Therefore, this PR introduces a method to ScheduleNode that returns
the function (more accurately, GlobalVar) currently being worked on.
With this we are able to know the function being worked on.
PR apache#14182 introduces the default GPU schedule pass, which makes uses
of the BlockCollector in Meta-Schedule. When requested, the
BlockCollector goes through every function in the input IRModule,
while in the case of applying default GPU schedule where we apply
schedule to one PrimFunc at a time, going through every PrimFunc is
wasteful.

Therefore, this PR enhances the BlockCollector so that it only focuses
on the function being worked on.
@tvm-bot
Copy link
Collaborator

tvm-bot commented Apr 11, 2023

Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from Reviewers by @-ing them in a comment.

Generated by tvm-bot

1 similar comment
@tvm-bot
Copy link
Collaborator

tvm-bot commented Apr 11, 2023

Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from Reviewers by @-ing them in a comment.

Generated by tvm-bot

@MasterJH5574
Copy link
Contributor Author

Depending on #14594

Copy link
Member

@junrushao junrushao left a comment

Choose a reason for hiding this comment

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

LGTM! Perhaps we should send both PRs to mainline.

@junrushao junrushao merged commit e416226 into apache:unity Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants