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

Fix __half for older architectures #1543

Merged
merged 2 commits into from
Mar 18, 2024

Conversation

miscco
Copy link
Collaborator

@miscco miscco commented Mar 15, 2024

Prior to SM53 not all mathematical functions are implemented for __half

This breaks internal testing . We must also properly qualify all functions, otherwise we might get conversion issues.

@miscco miscco requested review from a team as code owners March 15, 2024 07:49
@miscco miscco added libcu++ For all items related to libcu++ bug: functional backport branch/2.4.x For backporting to the 2.4.x release branch labels Mar 15, 2024
@miscco miscco changed the title Add missing qualifiers for half math functions Fix __half for older architectures Mar 15, 2024
Older architectures do not support all mathematical functions for `__half`

For that we provide backfills through a conversion to `float`

Fixes nvbug4563212
@miscco miscco merged commit 4ba5eb4 into NVIDIA:main Mar 18, 2024
586 checks passed
@miscco miscco deleted the properly_qualify_all_half_functions branch March 18, 2024 21:11
Copy link
Contributor

GitHub Actions is not permitted to create or approve pull requests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport branch/2.4.x For backporting to the 2.4.x release branch bug: functional libcu++ For all items related to libcu++
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants