Skip to content

Conversation

@masahi
Copy link
Member

@masahi masahi commented Dec 2, 2022

Currently, the condition for applying MultiLevelTiling is given by NeedsMultiLevelTiling function:

if (!NeedsMultiLevelTiling(sch->state(), sch->GetSRef(block_rv))) {

In practice, only multi-argument reduction operations like conv2d or dense are selected. I'm proposing to make this condition customizable for use cases where we might want to apply MultiLevelTiling to more ops (pooling etc).

@junrushao @csullivan @vinx13

@tvm-bot
Copy link
Collaborator

tvm-bot commented Dec 2, 2022

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

@masahi masahi changed the title [MetaSchedule] Make MultiLevelTiling application condition customizable [MetaSchedule] Make MultiLevelTiling apply condition customizable Dec 2, 2022
@masahi masahi force-pushed the ms-pool-cache-read branch from c0a9c98 to bce90b1 Compare December 2, 2022 06:49
@masahi masahi force-pushed the ms-pool-cache-read branch from bce90b1 to 9543fee Compare December 2, 2022 06:50
Copy link
Contributor

@csullivan csullivan left a comment

Choose a reason for hiding this comment

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

Thanks @masahi, LGTM.

@csullivan csullivan merged commit eb2781f into apache:main Dec 6, 2022
mikeseven pushed a commit to mikeseven/tvm that referenced this pull request Sep 27, 2023
…pache#13535)

Currently, the condition for applying MultiLevelTiling is given by NeedsMultiLevelTiling function and only multi-argument reduction operations like conv2d or dense are selected. This PR changes this condition to be customizable for use cases where we might want to apply MultiLevelTiling to more ops (pooling etc).

* define max pool on blocked layout

* introduce filter_fn to MultiLevelTiling
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.

4 participants