Skip to content

Fix optional operator* called on a disengaged value in clamp.cu#18655

Merged
rapids-bot[bot] merged 2 commits intorapidsai:branch-25.06from
davidwendt:fix-optional-clamp
May 6, 2025
Merged

Fix optional operator* called on a disengaged value in clamp.cu#18655
rapids-bot[bot] merged 2 commits intorapidsai:branch-25.06from
davidwendt:fix-optional-clamp

Conversation

@davidwendt
Copy link
Contributor

Description

Fixes invalid dereference of a cuda::std::optional with no value. The new CCCL version has assert checks for this which show up in a Debug build.

Error appears like this when running gtests/CLAMP_TEST --gtest_filter=ClampTestNumeric/0.InputNull:

/cudf/cpp/build/_deps/cccl-src/lib/cmake/libcudacxx/../../../libcudacxx/include/cuda/std/detail/libcxx/include/optional:867: operator*: block: [0,0,0], thread: [0,0,0] Assertion `optional operator* called on a disengaged value` failed.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@davidwendt davidwendt added bug Something isn't working 3 - Ready for Review Ready for review by team libcudf Affects libcudf (C++/CUDA) code. non-breaking Non-breaking change labels May 5, 2025
@davidwendt davidwendt self-assigned this May 5, 2025
@davidwendt davidwendt requested a review from a team as a code owner May 5, 2025 14:23
@davidwendt davidwendt requested review from bdice and mhaseeb123 May 5, 2025 14:23
@davidwendt
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit afe19de into rapidsai:branch-25.06 May 6, 2025
109 of 110 checks passed
@davidwendt davidwendt deleted the fix-optional-clamp branch May 6, 2025 14:16
vyasr added a commit to vyasr/cudf that referenced this pull request May 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 - Ready for Review Ready for review by team bug Something isn't working libcudf Affects libcudf (C++/CUDA) code. non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments