Skip to content

[libcu++] Replace cuda::__bad_any_cast with std::bad_any_cast#8799

Open
davebayer wants to merge 1 commit into
NVIDIA:mainfrom
davebayer:remove_bad_any_cast
Open

[libcu++] Replace cuda::__bad_any_cast with std::bad_any_cast#8799
davebayer wants to merge 1 commit into
NVIDIA:mainfrom
davebayer:remove_bad_any_cast

Conversation

@davebayer
Copy link
Copy Markdown
Contributor

All of the host standard libraries come with <any> header, thus we should just throw std::bad_any_cast instead of the internal cuda::__bad_any_cast.

@davebayer davebayer requested a review from a team as a code owner May 4, 2026 17:23
@davebayer davebayer requested a review from Jacobfaib May 4, 2026 17:23
@github-project-automation github-project-automation Bot moved this to Todo in CCCL May 4, 2026
@cccl-authenticator-app cccl-authenticator-app Bot moved this from Todo to In Review in CCCL May 4, 2026
@davebayer davebayer force-pushed the remove_bad_any_cast branch from 50898b3 to bd726d6 Compare May 4, 2026 17:25
Comment thread libcudacxx/include/cuda/__utility/__basic_any/basic_any_ref.h Outdated
@davebayer davebayer force-pushed the remove_bad_any_cast branch from bd726d6 to aaf4dba Compare May 4, 2026 17:31
@davebayer davebayer force-pushed the remove_bad_any_cast branch from aaf4dba to 8d69cbc Compare May 4, 2026 17:33
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

🥳 CI Workflow Results

🟩 Finished in 2h 21m: Pass: 100%/110 | Total: 2d 16h | Max: 1h 24m | Hits: 92%/316882

See results here.

#endif // no system header

#if _CCCL_HOSTED()
# include <any>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Did we check whether <typeinfo> would be smaller?

Copy link
Copy Markdown
Contributor

@miscco miscco left a comment

Choose a reason for hiding this comment

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

Question about which host header to include

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

3 participants