Skip to content

[CI] Enable sccache for multi-arch release builds (Linux + Windows)#5901

Draft
amd-chiranjeevi wants to merge 20 commits into
mainfrom
users/amd-chiranjeevi/enable_sccache_multi_arch
Draft

[CI] Enable sccache for multi-arch release builds (Linux + Windows)#5901
amd-chiranjeevi wants to merge 20 commits into
mainfrom
users/amd-chiranjeevi/enable_sccache_multi_arch

Conversation

@amd-chiranjeevi

@amd-chiranjeevi amd-chiranjeevi commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Motivation

Enables sccache for the multi-arch release build pipeline (TheRock core — not PyTorch),

workflow call chain:

multi_arch_release.yml
→ multi_arch_release_{linux,windows}.yml
→ multi_arch_build_{portable_linux,windows}.yml
→ multi_arch_build_{portable_linux,windows}_artifacts.yml

What changed

Value
Bucket therock-sccache-{dev,nightly,prerelease}
Region us-east-1
Account -
IAM role -
Key prefix (Linux) linux/multiarch/v1/
Key prefix (Windows) windows/multiarch/v1/
cache_type default sccache

Test Plan

to be updated

Test Result

to be updated

Submission Checklist

@amd-chiranjeevi amd-chiranjeevi added the ci:skip Skip all CI builds/tests for this PR label Jun 17, 2026
@amd-chiranjeevi amd-chiranjeevi force-pushed the users/amd-chiranjeevi/enable_sccache_multi_arch branch from 483cda6 to 7c222a9 Compare June 17, 2026 10:28
@amd-chiranjeevi

Copy link
Copy Markdown
Contributor Author

needed this patch ROCm/rocm-systems#6239 - without this patch getting errors when build hip-clr on windows log: https://github.com/ROCm/TheRock/actions/runs/27739979092/job/82100969694

@therock-pr-bot

therock-pr-bot Bot commented Jul 5, 2026

Copy link
Copy Markdown

❌ PR Check — Action Required

Check Status Details
🌿 Branch Name ✅ Pass
📝 PR Title/Description ❌ Fail Error: Title does not follow Conventional Commits style.
Expected: start with a valid type (feat, fix, docs, …).
Desired format: type(optional-scope): short description
Forbidden Files ✅ Pass
🧪 Unit Test ❌ Fail Error: Source/code files changed without an accompanying unit test.
Expected: add at least one test file named like test_<name>.py / test_<name>.cpp (or <name>_test.*).
Current: code file(s) changed: build_tools/setup_sccache_rocm.py; no test file found
🔎 pre-commit ❌ Fail Error: Check concluded with failure.
🚫 Draft PR 🔜 To Be Enabled
🚩 Feature Flag 🔜 To Be Enabled
📊 Code Coverage 🔜 To Be Enabled

⚠️ 3 policy check(s) failed. Please address the issues above before this PR can be Reviewed.

🚫 Please fix the failed policies

  • ❌ PR Title/Description
  • ❌ Unit Test
  • ❌ pre-commit

The Not ready to Review label was added to this PR. Once all policies pass, the label is removed automatically.

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

@therock-pr-bot

therock-pr-bot Bot commented Jul 5, 2026

Copy link
Copy Markdown

🚫 Please fix the failed policies before requesting reviews.

The following policy checks failed:

  • ❌ PR Title/Description
  • ❌ Unit Test

The Not ready to Review label has been added to this PR.
Once all policies pass, the label will be removed automatically.

@therock-pr-bot therock-pr-bot Bot added the Not ready to Review PR has unresolved policy failures — reviews blocked label Jul 6, 2026
chiranjeevipattigidi and others added 3 commits July 6, 2026 14:36
Adds a patch for rocm-libraries/rocwmma that removes the non-existent
-fallow-half-arguments-and-returns flag (passed via -Xclang) and the
unnecessary --driver-mode=g++ flag from CMAKE_CXX_FLAGS.

These flags caused sccache to mark all rocWMMA compilations as
non-cacheable ("Can't handle UnknownFlag arguments with -Xclang"),
producing ~1,041 cache misses per math-libs build. The flags are
dead — the clang flag does not exist in LLVM source, and rocBLAS
already removed it for the same reason.

Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:skip Skip all CI builds/tests for this PR Not ready to Review PR has unresolved policy failures — reviews blocked

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant