Skip to content

[SYCL][E2E] Re-enable WorkGroupMemory/basic_usage.cpp on HIP#22726

Open
zjin-lcf wants to merge 1 commit into
intel:syclfrom
zjin-lcf:sycl-hip-reenable-wgm-basic-usage
Open

[SYCL][E2E] Re-enable WorkGroupMemory/basic_usage.cpp on HIP#22726
zjin-lcf wants to merge 1 commit into
intel:syclfrom
zjin-lcf:sycl-hip-reenable-wgm-basic-usage

Conversation

@zjin-lcf

Copy link
Copy Markdown
Contributor

This test was marked UNSUPPORTED on HIP due to the flaky-CI hang tracked in #17339. The underlying failure mode on current ROCm is a compiler crash in GlobalOffsetPass ("Calling a function with a bad signature!") triggered by in-kernel asserts combined with get_global_id usage, which is fixed by #22677.

With that fix, the test compiles (asserts enabled) and passes on gfx942 (MI300A), so remove the UNSUPPORTED: hip marker.

Depends on #22677.

This test was marked UNSUPPORTED on HIP due to the flaky-CI hang tracked
in intel#17339. The underlying failure mode on current ROCm is a compiler
crash in GlobalOffsetPass ("Calling a function with a bad signature!")
triggered by in-kernel asserts combined with get_global_id usage, which
is fixed by intel#22677.

With that fix, the test compiles (asserts enabled) and passes on gfx942
(MI300A), so remove the UNSUPPORTED: hip marker.

Depends on intel#22677.
Closes intel#17339.

Co-authored-by: Cursor <cursoragent@cursor.com>
@zjin-lcf
zjin-lcf requested a review from a team as a code owner July 22, 2026 16:14

@KseniyaTikhomirova KseniyaTikhomirova left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, can be merged once dependent fix is merged, I see it has been already approved.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants