Skip to content

Update FindCUDNN.cmake for cuDNN 9#640

Merged
ksivaman merged 3 commits intoNVIDIA:mainfrom
cyanguwa:fix_cudnn_cmake
Jan 31, 2024
Merged

Update FindCUDNN.cmake for cuDNN 9#640
ksivaman merged 3 commits intoNVIDIA:mainfrom
cyanguwa:fix_cudnn_cmake

Conversation

@cyanguwa
Copy link
Collaborator

No description provided.

Signed-off-by: Charlene Yang <8636796+cyanguwa@users.noreply.github.com>
Copy link
Collaborator

@timmoon10 timmoon10 left a comment

Choose a reason for hiding this comment

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

Overall LGTM.

For future reference, this is taken from https://github.com/NVIDIA/cudnn-frontend/blob/main/cmake/cuDNN.cmake.

Signed-off-by: Charlene Yang <8636796+cyanguwa@users.noreply.github.com>
@cyanguwa
Copy link
Collaborator Author

/te-ci

@ptrendx
Copy link
Member

ptrendx commented Jan 30, 2024

Hmm, is there a possibility for us to instead actually use the one shipped with cuDNN frontend rather than shipping with our own?

@ksivaman ksivaman merged commit e2803b1 into NVIDIA:main Jan 31, 2024
ptrendx pushed a commit that referenced this pull request Feb 3, 2024
* update cudnn cmake for v9

Signed-off-by: Charlene Yang <8636796+cyanguwa@users.noreply.github.com>

* add back license information

Signed-off-by: Charlene Yang <8636796+cyanguwa@users.noreply.github.com>

---------

Signed-off-by: Charlene Yang <8636796+cyanguwa@users.noreply.github.com>
@cyanguwa cyanguwa deleted the fix_cudnn_cmake branch February 22, 2024 00:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants