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

[TOPI][CUDA] Support cuBLAS BatchMatMul #3936

Merged
merged 2 commits into from
Sep 12, 2019

Conversation

soiferj
Copy link
Contributor

@soiferj soiferj commented Sep 11, 2019

Add support for cuBLAS BatchMatMul if -libs=cublas is specified. This fills in a gap, since cublas is already supported for Dense. cblas is already supported for Dense and BatchMatMul on CPU.

@masahi @vinx13 would you be able to take a look?

Copy link
Member

@icemelon icemelon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you also add a test case?

topi/python/topi/cuda/batch_matmul.py Outdated Show resolved Hide resolved
@soiferj
Copy link
Contributor Author

soiferj commented Sep 11, 2019

Added a test case

@soiferj soiferj requested a review from icemelon September 11, 2019 19:46
Copy link
Member

@icemelon icemelon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@icemelon icemelon merged commit 88f9bfd into apache:master Sep 12, 2019
@icemelon
Copy link
Member

Thanks @soiferj

@soiferj soiferj deleted the soiferj/cublasbatch branch September 12, 2019 20:23
wweic pushed a commit to wweic/tvm that referenced this pull request Sep 16, 2019
* Support cuBLAS BatchMatMul

* Add test and check target name
wweic pushed a commit to wweic/tvm that referenced this pull request Sep 16, 2019
* Support cuBLAS BatchMatMul

* Add test and check target name
wweic pushed a commit to neo-ai/tvm that referenced this pull request Sep 16, 2019
* Support cuBLAS BatchMatMul

* Add test and check target name
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.

3 participants