Skip to content

Add support for building fatbin from clang ptx#368

Open
rbberger wants to merge 1 commit intoStanfordLegion:mainfrom
rbberger:rbberger/fix_clang_build
Open

Add support for building fatbin from clang ptx#368
rbberger wants to merge 1 commit intoStanfordLegion:mainfrom
rbberger:rbberger/fix_clang_build

Conversation

@rbberger
Copy link
Copy Markdown
Contributor

This enables building the embedded fatbinary using ptx generated by Clang. It is currently restricted to a single .cu file in $REALM_CUDA_SOURCES and a single $CMAKE_CUDA_ARCHITECTURES. The single .cu restriction is already the case also for the NVCC path anyway.

For more SMs per translation unit we would have to either ask CMake to fix their CUDA_PTX_COMPILATION support, or manually do it via loops and custom targets/commands. For the FleCSI use case both restrictions are currently sufficient.

@eddy16112 eddy16112 requested a review from muraj November 26, 2025 07:08
@codecov
Copy link
Copy Markdown

codecov bot commented Nov 26, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 27.74%. Comparing base (23ab773) to head (5166709).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #368      +/-   ##
==========================================
- Coverage   27.74%   27.74%   -0.01%     
==========================================
  Files         192      192              
  Lines       39502    39502              
  Branches    14382    14382              
==========================================
- Hits        10960    10959       -1     
  Misses      28151    28151              
- Partials      391      392       +1     

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

Signed-off-by: Richard Berger <rberger@lanl.gov>
@rbberger rbberger force-pushed the rbberger/fix_clang_build branch from 5166709 to cf17e6c Compare January 9, 2026 19:17
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.

2 participants