Skip to content

Conversation

@masahi
Copy link
Member

@masahi masahi commented Oct 13, 2022

Fix the VNNI test broken by #12895

And I've just ran rocm and vulkan dp4a auto tensorization tests locally and confirmed that they still worked.

@junrushao

@tvm-bot
Copy link
Collaborator

tvm-bot commented Oct 13, 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.

  • No users to tag found in teams: test See #10317 for details
  • Built docs for commit 92229c7 can be found here.

Generated by tvm-bot

@masahi masahi changed the title [TEST] Fix broken MetaSchedule tests [TEST] Fix the broken VNNI MetaSchedule test Oct 13, 2022
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!

@masahi
Copy link
Member Author

masahi commented Oct 14, 2022

@tvm-bot rerun

@Lunderberg
Copy link
Contributor

Thank you, @masahi, for this bugfix! It looks like it was causes quite a lot of CI failures.

@Lunderberg Lunderberg merged commit 605876e into apache:main Oct 14, 2022
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 10, 2022
* fixed MS vnni template-based tuning test

* enable local testing of dp4a auto tensorize

* fix rocm and vk auto-tensorize test

* tweaking to see why IS_IN_CI isn't working

* skip dp4a auto tensorize test since IS_IN_CI is not working

* fix broken hexagon test after onnx model update
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
* fixed MS vnni template-based tuning test

* enable local testing of dp4a auto tensorize

* fix rocm and vk auto-tensorize test

* tweaking to see why IS_IN_CI isn't working

* skip dp4a auto tensorize test since IS_IN_CI is not working

* fix broken hexagon test after onnx model update
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