[bjacob]

Move Debug arithmetic symbol checks into the existing rocjitsu-test-corpus
jobs and delete the standalone rocjitsu-arith-debug workflow.

The Release job runs the check with Clang; the GCC UBSan job runs it with GCC.
Each configures a separate, unsanitized Debug build, builds only rj_arith, and
runs rj_arith.baseline_linkage. No new jobs or simulator builds are added.

rj-arith compiles scalar and SIMD backends in separate translation units with
different CPU feature levels. Externally linkable inline helpers, including
standard-library template instantiations, must not share symbols across these
backends: the linker could retain a definition using unsupported instructions
and bypass CPU dispatch. Backend-local callable symbols need ISA identities.
Debug builds expose out-of-line definitions that optimization can hide; the
archive check verifies symbol isolation with both GCC and Clang.

Validation: GCC and Clang Debug library builds and symbol checks pass locally;
YAML and shell syntax checks pass. Existing corpus steps and matrix are unchanged.

Progress towards #13670.

[therock-pr-bot]

✅ All Checks Passed — Ready for Review

Check	Status	Details
📝 PR Description	✅ Pass	—
⛔ Forbidden Files	✅ Pass	—
🧪 Unit Test	✅ Pass	PR does not contain code files — Unit Test auto-passed
🔎 pre-commit	✅ Pass	—
🚫 Draft PR	🔜 To Be Enabled	—
🚩 Feature Flag	🔜 To Be Enabled	—
📊 Code Coverage	🔜 To Be Enabled	—
🤖 therock-pr-bot	✅ Pass	—

🎉 All checks passed! This PR is ready for review.

📖 Need help? See the Policy FAQ for details on every check and how to fix failures.

🙋 Wish to Override Policy?

[therock-pr-bot]

🎉 All checks passed! This PR is ready for review.

[kuhar]

I'd like this to be consolidated with existing job(s), not just the workflow. We shouldn't grow the number of jobs unless we have a very strong reason.

[bjacob]

Addressed: the Debug checks now run as steps in the existing Release (Clang) and GCC UBSan jobs. Each uses a separate unsanitized Debug build directory, builds only rj_arith, and runs the symbol-isolation check. Removed the separate job matrix and workflow. Both compiler builds and symbol checks pass locally.

[kuhar]

Can we move these new steps and constants to happen after the main build steps? I'd like to have it configured so that we go in the order of importance and allow it to fail fast: first regular build, then debug build, then run corpus tests.

[bjacob]

Moved the Debug arithmetic check immediately after the regular build step (which includes its CTest run), before the subsequent integration/corpus checks. Its compiler selections are now local to that Debug step rather than extra matrix fields. YAML and shell syntax checks pass.

[kuhar]

LGTM

[github-actions]

rocjitsu post-commit fuzz passed

The 32-worker multiplexer campaigns for commit 43c2680dba4d completed without recording a crash or hang.

• ASan+UBSan (6-minute): 612,371 input executions (1,738.28 inputs/s)
• TSan (6-minute): 515,626 input executions (1,474.87 inputs/s)

View the workflow run.

[systems-assistant]

TheRock Submodule Bump Activity

Newest first

• 2026-11-05 — Included in TheRock via ROCm/TheRock#9739.