Skip to content

Commit cec563e

Browse files
committed
Diagnostic: force sccache recache for modules build
Reintroduces test_on.cpp into the modular build (reverting the CMakeLists.txt change from 8f6735f) and sets SCCACHE_RECACHE=1 for the modules matrix entry. Goal: isolate whether the CI ICEs on this matrix entry (blocking merges on test_on.cpp in particular) are caused by a stale/corrupted sccache cache entry rather than a live, per-run compiler issue. This changes exactly one variable relative to the last known-failing config (e39bbcf, serialized -j1, test_on.cpp included, cache active): caching behavior for that one TU. If this passes: cache is implicated. Follow up by removing SCCACHE_RECACHE and rerunning to confirm the now-fresh entry is read back successfully, before deciding whether to leave RECACHE on permanently for this matrix entry. If this still ICEs: cache is ruled out as the (sole) cause; the CI failure is likely a genuine, deterministic issue in the CI Clang build (22.1.8, x86_64 Ubuntu) independent of caching.
1 parent 8f6735f commit cec563e

2 files changed

Lines changed: 5 additions & 1 deletion

File tree

.github/workflows/ci.cpu.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,10 @@ jobs:
7878
SCCACHE_DIST_URL: "https://amd64.linux.sccache.rapids.nvidia.com"
7979
SCCACHE_IDLE_TIMEOUT: "0"
8080
SCCACHE_NO_DIST_COMPILE: ${{ contains(matrix.name, 'modules') && '1' || '' }}
81+
# Diagnostic: force sccache to ignore/overwrite any existing cache entries for
82+
# the modules build, to rule out a stale/corrupted cached artifact as the cause
83+
# of the ICEs we've been seeing on this matrix entry (e.g. test_on.cpp).
84+
SCCACHE_RECACHE: ${{ contains(matrix.name, 'modules') && '1' || '' }}
8185
SCCACHE_REGION: "us-east-2"
8286
SCCACHE_S3_KEY_PREFIX: "nvidia-stdexec-dev"
8387
SCCACHE_S3_PREPROCESSOR_CACHE_KEY_PREFIX: "nvidia-stdexec-dev/preprocessor"

test/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ set(stdexec_test_sources
3232
stdexec/algos/adaptors/test_finally.cpp
3333
stdexec/algos/adaptors/test_into_variant.cpp
3434
stdexec/algos/adaptors/test_let_error.cpp
35+
stdexec/algos/adaptors/test_on.cpp
3536
stdexec/algos/adaptors/test_on3.cpp
3637
stdexec/algos/adaptors/test_sequence.cpp
3738
stdexec/algos/adaptors/test_starts_on.cpp
@@ -81,7 +82,6 @@ if(NOT STDEXEC_BUILD_MODULES)
8182
stdexec/algos/adaptors/test_bulk.cpp
8283
stdexec/algos/adaptors/test_let_stopped.cpp
8384
stdexec/algos/adaptors/test_let_value.cpp
84-
stdexec/algos/adaptors/test_on.cpp
8585
stdexec/algos/adaptors/test_on2.cpp
8686
stdexec/algos/adaptors/test_spawn_future.cpp
8787
stdexec/algos/adaptors/test_stopped_as_error.cpp

0 commit comments

Comments
 (0)