Skip to content

fix: remove sm_70 and sm_75 from default GPU architecture list - #17

Merged
gauravharsha merged 1 commit into
mainfrom
fix/remove-deprecated-gpu-archs
Jun 23, 2026
Merged

fix: remove sm_70 and sm_75 from default GPU architecture list#17
gauravharsha merged 1 commit into
mainfrom
fix/remove-deprecated-gpu-archs

Conversation

@egull

@egull egull commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Summary

  • sm_70 (Volta) and sm_75 (Turing) were dropped in CUDA 13. Having them as unconditional defaults in CudaArchitectures.cmake causes a hard build failure (nvcc fatal: Unsupported gpu architecture 'compute_70') on any system with CUDA 13+.
  • Removed both from the default list. The minimum default architecture is now sm_80 (Ampere, CUDA ≥ 11.0).
  • Users who require Volta/Turing support can still pass -DGPU_ARCHS="70;75" explicitly when using a CUDA 12 or older toolchain.

Test plan

  • Verify default build succeeds on CUDA 13+ without -DGPU_ARCHS
  • Verify -DGPU_ARCHS="70;75" still works on CUDA 12

🤖 Generated with Claude Code

sm_70 (Volta) and sm_75 (Turing) were dropped in CUDA 13. Keeping them
in the default list causes a hard build failure on any system with CUDA
13+. The minimum default is now sm_80 (Ampere), which is supported from
CUDA 11.0 onward. Users who need Volta/Turing support can still pass
-DGPU_ARCHS="70;75" explicitly with a CUDA 12 or older toolchain.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.70%. Comparing base (1c367bb) to head (360b725).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #17   +/-   ##
=======================================
  Coverage   96.70%   96.70%           
=======================================
  Files          13       13           
  Lines         850      850           
=======================================
  Hits          822      822           
  Misses         28       28           
Flag Coverage Δ
unittests 96.70% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gauravharsha
gauravharsha merged commit 23c530c into main Jun 23, 2026
1 check 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.

3 participants