Skip to content

fix: replace removed cudaDeviceProp::memoryClockRate for CUDA 13 compatibility - #16

Merged
gauravharsha merged 1 commit into
mainfrom
fix/cuda13-memclockrate
Jun 23, 2026
Merged

fix: replace removed cudaDeviceProp::memoryClockRate for CUDA 13 compatibility#16
gauravharsha merged 1 commit into
mainfrom
fix/cuda13-memclockrate

Conversation

@egull

@egull egull commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Summary

  • cudaDeviceProp::memoryClockRate was removed in CUDA 13, causing a compile error in cuda_check.cpp.
  • Replaced with cuDeviceGetAttribute(..., CU_DEVICE_ATTRIBUTE_MEMORY_CLOCK_RATE, i), which returns the same value in kHz and is available across all CUDA versions via the driver API (cuda.h is already included).

Test plan

  • Verify build succeeds with CUDA 13+
  • Verify reported peak memory bandwidth value is unchanged compared to the old memoryClockRate path on CUDA 12

🤖 Generated with Claude Code

…atibility

cudaDeviceProp::memoryClockRate was removed in CUDA 13. Use the driver
API cuDeviceGetAttribute with CU_DEVICE_ATTRIBUTE_MEMORY_CLOCK_RATE
instead, which returns the same value in kHz and is available across all
CUDA versions.

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.71%. Comparing base (1c367bb) to head (2a4bc34).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #16   +/-   ##
=======================================
  Coverage   96.70%   96.71%           
=======================================
  Files          13       13           
  Lines         850      852    +2     
=======================================
+ Hits          822      824    +2     
  Misses         28       28           
Flag Coverage Δ
unittests 96.71% <100.00%> (+<0.01%) ⬆️

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 43abdf3 into main Jun 23, 2026
1 check passed
@egull
egull deleted the fix/cuda13-memclockrate branch June 23, 2026 18:42
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