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 availability of is_constant_evaluated on old MSVC #1180

Merged

Conversation

miscco
Copy link
Collaborator

@miscco miscco commented Dec 6, 2023

@ptheywood found that there is an unfortunate incompatability of MSVC2019 which does support __builtin_is_constant_evaluated from 19.24 onwards and old nvcc below 11.3 which does not.

Fix this by hardening the condition we use to detect availability

Fixes #1179

@ptheywood found that there is an unfortunate incompatability of MSVC2019 which does support `__builtin_is_constant_evaluated` from 19.24 onwards and old nvcc below 11.3 which does not.

Fix this by hardening the condition we use to detect availability

Fixes NVIDIA#1179
@miscco miscco requested review from a team as code owners December 6, 2023 12:53
@miscco miscco added libcu++ For all items related to libcu++ backport branch/2.3.x For backporting to the 2.3.x release branch bug: functional labels Dec 6, 2023
@jrhemstad jrhemstad merged commit 6540f9f into NVIDIA:main Dec 8, 2023
517 checks passed
Copy link
Contributor

github-actions bot commented Dec 8, 2023

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

@miscco miscco deleted the fix_is_constant_evaluated_old_msvc_old_nvcc branch December 8, 2023 17:09
miscco added a commit to miscco/cccl that referenced this pull request Dec 8, 2023
@ptheywood found that there is an unfortunate incompatability of MSVC2019 which does support `__builtin_is_constant_evaluated` from 19.24 onwards and old nvcc below 11.3 which does not.

Fix this by hardening the condition we use to detect availability

Fixes NVIDIA#1179
jrhemstad pushed a commit that referenced this pull request Dec 13, 2023
@ptheywood found that there is an unfortunate incompatability of MSVC2019 which does support `__builtin_is_constant_evaluated` from 19.24 onwards and old nvcc below 11.3 which does not.

Fix this by hardening the condition we use to detect availability

Fixes #1179
@miscco miscco added backport done This PR has been backported to the relevant branch and removed backport branch/2.3.x For backporting to the 2.3.x release branch labels Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport done This PR has been backported to the relevant branch bug: functional libcu++ For all items related to libcu++
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[BUG]: Visual Studio 2019 + CUDA <= 11.2 __builtin_is_constant_evaluated compilation error
2 participants