Commit 61364d6
committed
Add build-only ROCm CI job
The CUDA CI job has been failing on every push to main since at least 2026-02-23 with an unrelated gcc-13/CUDA-12.1 compiler-version mismatch — pre-existing breakage, not introduced by this PR. The Scheduled workflow (different toolchain pin) still passes on main, which is how releases get cut.
This commit adds a parallel `linux_rocm_build` job that catches build regressions in the ROCm/HIP code paths added by this PR (USE_ROCM guards, hipify-touched sources, the pulsar `_rn`/`atomicAdd_block` intrinsic replacements). It runs inside the official `rocm/dev-ubuntu-22.04:6.2.4` container on a CPU-only `ubuntu-latest` runner: AMD GPU hardware isn't required to *compile* against ROCm, only to run tests, so we skip test execution and keep CI minutes low. The container ships hipcc plus the ROCm SDK at /opt/rocm (auto-detected as ROCM_HOME). We pip-install torch==2.4.1 from the rocm6.2 wheel index, verify torch.version.hip is set, then `pip install -e .` with FORCE_CUDA=1 (the runner has no GPU, so torch.cuda.is_available() is False and FORCE_CUDA flips the CUDAExtension branch on). A final smoke import confirms `_C.PulsarRenderer` linked.
ROCm 6.2 was chosen because pulsar's port uses `__hip_atomic_fetch_add` with `__HIP_MEMORY_SCOPE_WORKGROUP` and the modern HIP atomic API, which is stable in 6.2+ (matches the TODO comment that Suresh Babu Kolla left at pulsar/gpu/commands.h:63 referencing ROCM-6.2 as the minimum target).
The CUDA job is left unchanged.
Co-Authored-By: Claude Opus 4.7 (1M context)1 parent eea2762 commit 61364d6
1 file changed
Lines changed: 35 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
0 commit comments