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

VS2022 17.11 / MSVC 1941 CI Fix #137

Merged
merged 2 commits into from
Sep 4, 2024
Merged

VS2022 17.11 / MSVC 1941 CI Fix #137

merged 2 commits into from
Sep 4, 2024

Conversation

ptheywood
Copy link
Member

VS2022 17.11 / MSVC 19.41 include a stdlib change which enforces CUDA >= 12.4 in a static assert (which I can't find a way to ignore during CMake language detection).

This Adds a new warning if this is why cuda hasn't been found, ensures the warning is triggered, and updates CI to not use VS2022 with CUDA < 12.4

@ptheywood ptheywood force-pushed the msvc1711-ci-fix branch 3 times, most recently from 7f12a46 to 2bf9c59 Compare August 30, 2024 11:33
Also moves where CheckCompilerFunctionality is included to ensure it is triggered before CUDA is attempted
@ptheywood ptheywood requested a review from Robadob August 30, 2024 12:00
@ptheywood ptheywood marked this pull request as ready for review August 30, 2024 12:00
@ptheywood ptheywood merged commit 09f25de into master Sep 4, 2024
14 checks passed
@ptheywood ptheywood deleted the msvc1711-ci-fix branch September 4, 2024 10:48
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