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

Add [[noreturn]] to set_error_and_throw_exception #55

Closed
wants to merge 1 commit into from

Conversation

r-barnes
Copy link

Add [[noreturn]] to set_error_and_throw_exception

Add `[[noreturn]]` to `set_error_and_throw_exception`
Anerudhan added a commit that referenced this pull request Jan 31, 2024
[Bug fix] Fixed an issue where in some cases with padding the stats of SDPA forward node where having NANs.

[Bug fix] In some older cuda toolkits, eg. cuda 11.4, float to half conversion is not implicit. This was raised in [PR-57](<#57>). Thanks @drisspg for reporting this. A more correct fix has been implemented in this patch.

[Enhancement] Accepting github [PR-55](<#55>). Thanks @r-barnes for the suggestion.
Anerudhan added a commit that referenced this pull request Jan 31, 2024
[Bug fix] Fixed an issue where in some cases with padding the stats of SDPA forward node where having NANs.

[Bug fix] In some older cuda toolkits, eg. cuda 11.4, float to half conversion is not implicit. This was raised in [PR-57](<#57>). Thanks @drisspg for reporting this. A more correct fix has been implemented in this patch.

[Enhancement] Accepting github [PR-55](<#55>). Thanks @r-barnes for the suggestion.
@Anerudhan Anerudhan mentioned this pull request Jan 31, 2024
Anerudhan added a commit that referenced this pull request Jan 31, 2024
[Bug fix] Fixed an issue where in some cases with padding the stats of SDPA forward node where having NANs.

[Bug fix] In some older cuda toolkits, eg. cuda 11.4, float to half conversion is not implicit. This was raised in [PR-57](<#57>). Thanks @drisspg for reporting this. A more correct fix has been implemented in this patch.

[Enhancement] Accepting github [PR-55](<#55>). Thanks @r-barnes for the suggestion.
@Anerudhan
Copy link
Collaborator

Hi @r-barnes,

Thank you for the message. cuDNN team has added the changes in 1.0.3

At this point we do not accept external PRs directly.

Please re-open this if you need.

Thanks

@Anerudhan Anerudhan closed this Jan 31, 2024
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.

2 participants