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

PR #18840: [NVIDIA] Support larger head dim for cudnn fmha #19828

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

copybara-service[bot]
Copy link

@copybara-service copybara-service bot commented Nov 26, 2024

PR #18840: [NVIDIA] Support larger head dim for cudnn fmha

Imported from GitHub PR #18840

Since cudnn v9.5.0, the larger head dim of 256 is supported. This PR enables this improvement.

cc @Cjkkkk
Copybara import of the project:

--
723af68 by kaixih [email protected]:

Support larger head dim for cudnn fmha

--
5177dbd by kaixih [email protected]:

Add unit test

--
9e8d8be by kaixih [email protected]:

Formatting

--
5d93712 by kaixih [email protected]:

Address comments

--
02777e1 by kaixih [email protected]:

Separate tests

--
e4cc8eb by kaixih [email protected]:

Clang format

Merging this change closes #18840

FUTURE_COPYBARA_INTEGRATE_REVIEW=#18840 from kaixih:cudnn_fmha_large_head_dim e4cc8eb

@copybara-service copybara-service bot force-pushed the test_700218013 branch 3 times, most recently from e80b34f to fb4946b Compare November 26, 2024 08:43
Imported from GitHub PR #18840

Since [cudnn v9.5.0](https://docs.nvidia.com/deeplearning/cudnn/latest/release-notes.html#cudnn-9-5-0), the larger head dim of 256 is supported. This PR enables this improvement.

cc @Cjkkkk
Copybara import of the project:

--
723af68 by kaixih <[email protected]>:

Support larger head dim for cudnn fmha

--
5177dbd by kaixih <[email protected]>:

Add unit test

--
9e8d8be by kaixih <[email protected]>:

Formatting

--
5d93712 by kaixih <[email protected]>:

Address comments

--
02777e1 by kaixih <[email protected]>:

Separate tests

--
e4cc8eb by kaixih <[email protected]>:

Clang format

Merging this change closes #18840

FUTURE_COPYBARA_INTEGRATE_REVIEW=#18840 from kaixih:cudnn_fmha_large_head_dim e4cc8eb
PiperOrigin-RevId: 700218013
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.

1 participant