Skip to content

Workaround for new cuSparse API introduced in CUDA patch release #3057

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

Merged
merged 1 commit into from
Dec 2, 2020

Conversation

9prady9
Copy link
Member

@9prady9 9prady9 commented Dec 2, 2020

Description

New API of cuSparse was introduced in 10.1.168 for Linux and the older
10.1.105 version doesn't it.

Unfortunately, when the new API was introduced in ArrayFire's code base,
I was testing against versions 10.1.168 or newer and hence didn't realize
that this new API was introduced in a patch/fix release - unconventional.

This change enables the new API only from 10.2.* on Linux since CUDA toolkit
version variable set by CMake doesn't provide patch number.

Changes to Users

Developers building ArrayFire's CUDA backend using toolkit version 10.1.105 will be able to build fine now.

Checklist

  • Rebased on latest master
  • Code compiles
  • [ ] Tests pass
  • [ ] Functions added to unified API
  • [ ] Functions documented

New API of cuSparse was introduced in 10.1.168 for Linux and the older
10.1.105 version doesn't it.

Unfortunately, when the new API was introduced in ArrayFire's code base,
I was testing against versions 10.1.168 or newer and hence didn't realize
that this new API was introduced in a patch/fix release - unconventional.

This change enables the new API only from 10.2.* on Linux since CUDA toolkit
version variable set by CMake doesn't provide patch number.
@9prady9 9prady9 merged commit 28f286b into arrayfire:master Dec 2, 2020
@9prady9 9prady9 deleted the workaround_cusparse_missing_api branch December 2, 2020 16:58
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