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

Enable miopen transpose convolution and fp16 support #3952

Merged
merged 2 commits into from
Sep 15, 2019

Conversation

petrex
Copy link
Contributor

@petrex petrex commented Sep 13, 2019

This PR enables miopen support for

  • Transpose Convolution
  • FP16 data type

Please also expect follow-up PRs for

  • More Cconvolution mode (Group/Depthwise for example)

  • More data type (INT8, Bfloat16 etc)

  • Optimal schedule and more test coverage for AMD intrinsic.

@petrex
Copy link
Contributor Author

petrex commented Sep 13, 2019

Thanks @masahi ! Appreciate your feedback and support.

@masahi
Copy link
Member

masahi commented Sep 13, 2019

@petrex can you fix the spacing issue? It seems the diff is larger than necessary.

@petrex
Copy link
Contributor Author

petrex commented Sep 13, 2019

yes, let me fix that so you can review it easily

@petrex
Copy link
Contributor Author

petrex commented Sep 14, 2019

@masahi Here's the diff against master. Thanks

@masahi masahi merged commit 9e4f07b into apache:master Sep 15, 2019
wweic pushed a commit to wweic/tvm that referenced this pull request Sep 16, 2019
* Enable miopen transpose convolution and fp16 support

* linter
wweic pushed a commit to wweic/tvm that referenced this pull request Sep 16, 2019
* Enable miopen transpose convolution and fp16 support

* linter
wweic pushed a commit to neo-ai/tvm that referenced this pull request Sep 16, 2019
* Enable miopen transpose convolution and fp16 support

* linter
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