Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MetaSchedule] Enhance Conv2d NCHW Winograd Schedule Rules #12127

Merged
merged 15 commits into from
Aug 3, 2022

Conversation

zxybazh
Copy link
Member

@zxybazh zxybazh commented Jul 19, 2022

This PR introduces some new trials to expand the search space for Winograd Conv2d Workload by writing block level schedule rules. Tests will also be included.

@zxybazh zxybazh force-pushed the feature/2022-07-18/winograd-cuda branch 2 times, most recently from d749c97 to f0826f6 Compare August 1, 2022 21:28
@zxybazh
Copy link
Member Author

zxybazh commented Aug 2, 2022

@tvm-bot rerun

@zxybazh zxybazh force-pushed the feature/2022-07-18/winograd-cuda branch from da88554 to 0bb5078 Compare August 2, 2022 07:07
@zxybazh zxybazh marked this pull request as ready for review August 2, 2022 07:07
@zxybazh zxybazh changed the title [MetaSchedule] Enhance Conv2d Winograd Schedule Rules [MetaSchedule] Enhance Conv2d NCHW Winograd Schedule Rules Aug 2, 2022
@zxybazh
Copy link
Member Author

zxybazh commented Aug 2, 2022

Hi Wuwei, I have removed bgemm block schedule and use MultiLevelTiling instead. It reproduced same performance in conv2d test, I’m running e2e tests as well. With other issues also addressed, would you please take another look? Thanks!

@zxybazh zxybazh merged commit 46a8498 into apache:main Aug 3, 2022
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
)

* Update winograd schedule rules.

* Remove extra part for setting local storage scope.

* Fix bgemm schedule.

* Add winograd tile size to annotation.

* Finish winograd schedule rules.

* Process add relu.

* Modify to nchw rules.

* Add missing nchw output rules.

* Add winograd conv2d nchw search space test.

* Fix lint.

* Leave consumer of output to autoinline.

* Remove bgemm rules.

* Remove bgemm schedule rule annotation.

* Update unit test.

* Fix test case.
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.

2 participants