diff --git a/tools/compile_overlays.py b/tools/compile_overlays.py index 09ba2016..8ce786e8 100644 --- a/tools/compile_overlays.py +++ b/tools/compile_overlays.py @@ -3255,6 +3255,15 @@ def current_variant_func_id_coverage(func_ids: list, data: bytes, # contract/header change (which bumps the hash -> fresh dir -> fresh memo) # re-attempts everything (e.g. the psx_rfe_mark_escape contract fix). # So a memoized skip only ever elides a build that is deterministically doomed. +# Printed when a capture contributes no walk-root seed. The wording is +# deliberate: the region is NOT declared data-only -- its executed dispatch +# demands are still served by the isolated-fragment pass, which runs +# separately. Named so the two emit sites and the test cannot drift apart. +# They did: 4683e923 reworded both prints and left +# test_compile_overlays_static_split asserting the old string. +NO_SHARED_WALK_ROOT_SEEDS_SKIP = ( + 'SKIP: no shared walk-root seeds; checking fragments separately') + INTERIOR_FAIL_MEMO = 'interior_fail_memo.txt' @@ -5818,7 +5827,7 @@ def _static_capture_job(cap, args, toml, forced_interiors, static_out, result): seed.split()[0].startswith('0x')) ] if not root_seeds: - print(' SKIP: no shared walk-root seeds; checking fragments separately\n') + print(f' {NO_SHARED_WALK_ROOT_SEEDS_SKIP}\n') result['outcome'] = 'skip' return @@ -6366,7 +6375,7 @@ def main(): seed.split()[0].startswith('0x')) ] if not root_seeds: - print(' SKIP: no shared walk-root seeds; checking fragments separately\n') + print(f' {NO_SHARED_WALK_ROOT_SEEDS_SKIP}\n') stats.add_skip() return diff --git a/tools/test_compile_overlays_static_split.py b/tools/test_compile_overlays_static_split.py index 0d9f8c3f..1c67d16c 100644 --- a/tools/test_compile_overlays_static_split.py +++ b/tools/test_compile_overlays_static_split.py @@ -227,7 +227,11 @@ class StaticWorkerContractTests(unittest.TestCase): 'overlays_static.c') self.assertEqual(res['outcome'], 'skip') self.assertIsNone(res['part']) - self.assertIn('SKIP: no walk-root seeds', res['log']) + # Assert the constant, not a copy of its text: this line asserted a + # stale literal from 4683e923 until 2026-09-18 and the test was red + # on master the whole time. + self.assertIn(compile_overlays.NO_SHARED_WALK_ROOT_SEEDS_SKIP, + res['log']) self.assertEqual(res['requested_entries'], set())