Skip to content

[SYCL][Graph] Mark re-enabled Graph Arc tests as unsupported #11611

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 20, 2023

Conversation

EwanC
Copy link
Contributor

@EwanC EwanC commented Oct 20, 2023

PR #11602 renabled some Graph test-e2e tests which started passing on Arc in post-commit CI after the merge of #11565. However, these passes are not guaranteed to be deterministic. See post-commit https://github.com/intel/llvm/actions/runs/6580530144/job/17879085527 run where Graph/RecordReplay/buffer_copy_host2target_offset.cpp failed after being renabled.

Mark the tests reenabled by #11602 as unsupported to keep post-commit CI clean by skipping the tests on Arc, until support for the Arc target in the Graph tests can be investigated properly.

PR intel#11602 renabled some Graph test-e2e tests which started
passing on Arc in post-commit CI. However, these passes
are guaranteed to be deterministic. See post-commit
https://github.com/intel/llvm/actions/runs/6580530144/job/17879085527
run where `Graph/RecordReplay/buffer_copy_host2target_offset.cpp`
failed.

Mark the reenabled tests as unsupported to keep post-commit CI
clean, until support for the Arc target can be investigated
properly.
@EwanC EwanC requested a review from a team as a code owner October 20, 2023 07:29
@EwanC EwanC temporarily deployed to WindowsCILock October 20, 2023 07:30 — with GitHub Actions Inactive
@EwanC EwanC temporarily deployed to WindowsCILock October 20, 2023 07:42 — with GitHub Actions Inactive
@againull againull merged commit ef70eeb into intel:sycl Oct 20, 2023
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.

3 participants