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

Early checking added and new test cases added for schedule fuse #5010

Merged
merged 5 commits into from
Mar 10, 2020

Conversation

ANSHUMAN87
Copy link
Contributor

@ANSHUMAN87 ANSHUMAN87 commented Mar 9, 2020

PR contains
1:> Early checking for inner & outer position orderings.
2:> New test case for the split+fuse and failure scenarios with reordering added.

@ANSHUMAN87 ANSHUMAN87 changed the title Test case fuse updated Early checking added and new test cases added for schedule fuse Mar 9, 2020
@ANSHUMAN87
Copy link
Contributor Author

@tqchen , @MarisaKirisame, @zhiics

@zhiics
Copy link
Member

zhiics commented Mar 9, 2020

Could you add some description about what problems are trying to solve? It would help people to understand the purpose of the PR.

@ANSHUMAN87
Copy link
Contributor Author

Could you add some description about what problems are trying to solve? It would help people to understand the purpose of the PR.

@zhiics : Thanks for your feedback! It is done now!

@zhiics zhiics merged commit c41e040 into apache:master Mar 10, 2020
@zhiics
Copy link
Member

zhiics commented Mar 10, 2020

Thanks @ANSHUMAN87

trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Apr 16, 2020
…he#5010)

* [1] New test case added for fuse

* [2] New test case added for fuse

* [3] New test case added for fuse

* [4] New test case added for fuse

* [5] Early check added
zhiics pushed a commit to neo-ai/tvm that referenced this pull request Apr 17, 2020
…he#5010)

* [1] New test case added for fuse

* [2] New test case added for fuse

* [3] New test case added for fuse

* [4] New test case added for fuse

* [5] Early check added
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.

2 participants