Skip to content

[AutoScheduler] Fix mutate auto unroll #6807

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.

Sign up for GitHub

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

Jump to bottom
Merged
merrymercy merged 1 commit into from
Oct 31, 2020
Merged

[AutoScheduler] Fix mutate auto unroll #6807

merrymercy merged 1 commit into from
Oct 31, 2020

Conversation

@merrymercy
Copy link
Member

@merrymercy merrymercy commented Oct 31, 2020

  • Fix mutate auto unroll. The previous implementation forgets to update the stage->attrs, which will make ToStr() wrong and lead to duplicated measurements
  • Do not import the RPC module on the top-level to remove the dependency of tornado

@merrymercy merrymercy mentioned this pull request Oct 31, 2020
Merged
2 tasks
@merrymercy merrymercy requested a review from comaniac October 31, 2020 01:47
@merrymercy
Copy link
Member Author

merrymercy commented Oct 31, 2020

cc @jcf94 @FrozenGene

@merrymercy merrymercy merged commit 2625866 into apache:main Oct 31, 2020
@merrymercy merrymercy deleted the pr-fix-auto-unroll branch October 31, 2020 10:02
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Dec 2, 2020
@merrymercy
Fix mutate auto unroll (apache#6807)
d5ed7e8
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Dec 4, 2020
@merrymercy
Fix mutate auto unroll (apache#6807)
2a1bae5
trevor-m pushed a commit to neo-ai/tvm that referenced this pull request Dec 4, 2020
@merrymercy @trevor-m
Fix mutate auto unroll (apache#6807)
7f21783
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tqchen tqchen tqchen approved these changes

@comaniac comaniac comaniac approved these changes

Assignees

No one assigned

Labels

status: need backport

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@merrymercy @tqchen @comaniac