Skip to content

Align and minimize cudarc feature contracts#11

Merged
shinaoka merged 4 commits into
mainfrom
codex/cudarc-feature-unification
Jul 17, 2026
Merged

Align and minimize cudarc feature contracts#11
shinaoka merged 4 commits into
mainfrom
codex/cudarc-feature-unification

Conversation

@shinaoka

@shinaoka shinaoka commented Jul 17, 2026

Copy link
Copy Markdown
Member

Summary

  • align the normal CUDA dependency with tenferro's explicit CUDA 12.8 and dynamic-loading contract
  • remove build-system CUDA detection and fallback loading/version features
  • keep the build dependency role-minimal (std, driver, dynamic-loading, cuda-12080)
  • preserve runtime, NVRTC, and NCCL on the normal dependency
  • add contract tests for both dependency roles

Why

Cargo always builds normal and build dependencies as separate units. Giving both units the full normal feature set did not merge them and increased a controlled downstream release target by about 11 MiB. The build script only reads cudarc::driver::sys::CUDA_VERSION, so its dependency should share the explicit CUDA floor/loading mode without inheriting runtime, NVRTC, or NCCL.

The final role-minimal contract reduced the same downstream target by about 3.7 MiB versus the old fallback/detection configuration. NCCL remains required because CubeCL CUDA communication currently uses communicator, all-reduce, send, and receive APIs unconditionally.

Validation

  • cargo fmt --all --check
  • standalone cudarc feature contract tests
  • CUDARC_CUDA_VERSION=12080 cargo check -p t4a-cubecl-cuda
  • tenferro CUDA consumer check against revision 072cf82387ae099ce1d23efe6c0cbc68e231d9d2
  • fresh controlled downstream release/debug measurements

@shinaoka shinaoka changed the title Align cudarc features with CUDA 12.8 runtime Align and minimize cudarc feature contracts Jul 17, 2026
@shinaoka
shinaoka marked this pull request as ready for review July 17, 2026 06:43
@shinaoka
shinaoka merged commit b9e8f0f into main Jul 17, 2026
6 of 9 checks passed
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.

1 participant