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 MSVC issues #1261

Merged
merged 2 commits into from
Jan 17, 2024
Merged

Fix MSVC issues #1261

merged 2 commits into from
Jan 17, 2024

Conversation

miscco
Copy link
Collaborator

@miscco miscco commented Jan 10, 2024

This addresses various fixes for MSVC 19.38

Those are mostly test fixes, but also a product code change that ensures that we are able to emplace into an optional with a const qualified base type

Addresses nvbug3843240

For some reasons CI is acting up parsing some macoss specific XFAILs

We do not support that, so just remove them
@miscco miscco requested review from a team as code owners January 10, 2024 12:48
@miscco miscco changed the title Fix_various_nvbugs Fix MSVC issues Jan 10, 2024
@miscco miscco added nvbug Has an associated internal NVIDIA NVBug. libcu++ For all items related to libcu++ backport branch/2.3.x For backporting to the 2.3.x release branch bug: functional labels Jan 10, 2024
@@ -27,7 +27,7 @@

_LIBCUDACXX_BEGIN_NAMESPACE_STD

template<class _Func, class... _Args, class = decltype(std::declval<_Func>()(std::declval<_Args>()...))>
template<class _Func, class... _Args, class = decltype(_CUDA_VSTD::declval<_Func>()(_CUDA_VSTD::declval<_Args>()...))>
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙀

@miscco miscco merged commit 5243a84 into NVIDIA:main Jan 17, 2024
538 checks passed
@miscco miscco deleted the fix_various_nvbugs branch January 17, 2024 07:29
Copy link
Contributor

Backport failed for branch/2.3.x, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally.

git fetch origin branch/2.3.x
git worktree add -d .worktree/backport-1261-to-branch/2.3.x origin/branch/2.3.x
cd .worktree/backport-1261-to-branch/2.3.x
git checkout -b backport-1261-to-branch/2.3.x
ancref=$(git merge-base 2771c61545eff4ec3cede24ba7963c4eebc4bbaf a7b9d41c3b26f90dac1a7120091b5ce9adab0ac2)
git cherry-pick -x $ancref..a7b9d41c3b26f90dac1a7120091b5ce9adab0ac2

miscco added a commit to miscco/cccl that referenced this pull request Jan 18, 2024
For some reasons CI is acting up parsing some macoss specific XFAILs

We do not support that, so just remove them

Fix an issue with MSVC not being able to call `construct_at` on a const qualified type

Addresses nvbug3843240
jrhemstad pushed a commit that referenced this pull request Jan 30, 2024
For some reasons CI is acting up parsing some macoss specific XFAILs

We do not support that, so just remove them

Fix an issue with MSVC not being able to call `construct_at` on a const qualified type

Addresses nvbug3843240
@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 Feb 22, 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++ nvbug Has an associated internal NVIDIA NVBug.
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants