Skip to content

Skip ptx-json tests for clang-cuda#5841

Merged
davebayer merged 1 commit intoNVIDIA:mainfrom
davebayer:skip_ptx_json_tests_for_clang_cuda
Sep 10, 2025
Merged

Skip ptx-json tests for clang-cuda#5841
davebayer merged 1 commit intoNVIDIA:mainfrom
davebayer:skip_ptx_json_tests_for_clang_cuda

Conversation

@davebayer
Copy link
Contributor

This PR skips the ptx-json tests for clang-cuda.

clang-cuda doesn't implement the "C" inline PTX constraint and it causes clang 20 to fail the compilation. Even nvcc and nvrtc support it only since 12.5.

On top of that I remove the need storage_helper type which is not needed.

@copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Sep 10, 2025

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@cccl-authenticator-app cccl-authenticator-app bot moved this from Todo to In Progress in CCCL Sep 10, 2025
@davebayer
Copy link
Contributor Author

/ok to test dd9a3ef

@davebayer davebayer marked this pull request as ready for review September 10, 2025 10:59
@davebayer davebayer requested review from a team as code owners September 10, 2025 10:59
@davebayer davebayer requested a review from miscco September 10, 2025 10:59
@cccl-authenticator-app cccl-authenticator-app bot moved this from In Progress to In Review in CCCL Sep 10, 2025
@github-actions
Copy link
Contributor

🥳 CI Workflow Results

🟩 Finished in 2h 15m: Pass: 100%/176 | Total: 1d 19h | Max: 1h 11m | Hits: 99%/177576

See results here.

@davebayer davebayer enabled auto-merge (squash) September 10, 2025 12:29
@davebayer davebayer merged commit f1f83b8 into NVIDIA:main Sep 10, 2025
189 checks passed
@github-project-automation github-project-automation bot moved this from In Review to Done in CCCL Sep 10, 2025
pciolkosz pushed a commit to pciolkosz/cccl that referenced this pull request Sep 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants