Skip to content

hipthreads: Disable hipthreads failures blocked by Compiler ww28 SMP 2.5 - #7158

Draft
lajagapp wants to merge 1 commit into
mainfrom
users/lajagapp/disable-hipthread-test
Draft

hipthreads: Disable hipthreads failures blocked by Compiler ww28 SMP 2.5#7158
lajagapp wants to merge 1 commit into
mainfrom
users/lajagapp/disable-hipthread-test

Conversation

@lajagapp

@lajagapp lajagapp commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Summary

Temporarily excludes hipthreads failures that block the Compiler ww28 SMP 2.5 promotion in TheRock#7052 (amd-llvm 7287e2062fa6). Linux and Windows jobs fail for unrelated reasons, so each uses the mechanism that matches its failure mode.
Tracked by #7157.

Changes

test_hipthreads.py excludes create_late.pass.cpp on Linux, fetch_test_configurations.py marks the Windows hipthreads job as expected-failure, and tests/fetch_test_configurations_test.py covers the new matrix field.

Linux

create_late.pass.cpp is marked upstream as "expected to fail", but it passes with the new compiler, and lit fails the run on that unexpected pass. It is the only test that changed out of 384.

It is skipped with lit --filter-out. The alternative, --xfail-not, would break main, where the old compiler still makes the test fail as expected.

Windows

lit builds a small compiler-check program before running anything, and that build now fails at link time (lld-link: error: relocation against symbol in discarded section: __start_llvm_offload_entries). No tests run at all, so there is nothing to skip and the whole job is marked as expected to fail.

The xfail comes from a new expect_failure_platform field on the hipthreads matrix entry rather than a hardcoded check in the job loop, following the existing exclude_family style.

Related

Signed-off-by: Lenine Ajagappane <Lenine.Ajagappane@amd.com>
@lajagapp
lajagapp force-pushed the users/lajagapp/disable-hipthread-test branch from 90ae4d7 to 1db049f Compare August 6, 2026 17:06
@therock-pr-bot

therock-pr-bot Bot commented Aug 6, 2026

Copy link
Copy Markdown

✅ All Checks Passed — Ready for Review

Check Status Details
📝 PR Description ✅ Pass
Forbidden Files ✅ Pass
🧪 Unit Test ✅ Pass
🔎 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

therock-pr-bot Bot commented Aug 6, 2026

Copy link
Copy Markdown

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: TODO

Development

Successfully merging this pull request may close these issues.

2 participants