Skip to content

[SYCL][Graph[E2E] Add an extra test to invoke copy engine on L0 #17324

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
Mar 11, 2025

Conversation

konradkusiak97
Copy link
Contributor

Following #16830, this PR adds an additional e2e graph test to invoke copy engine on L0 devices and test its behaviour.

@konradkusiak97 konradkusiak97 force-pushed the Adde2eTestForCopyEngine branch from 92dfb42 to 4557dde Compare March 6, 2025 00:14
@konradkusiak97 konradkusiak97 force-pushed the Adde2eTestForCopyEngine branch from 4557dde to fff0709 Compare March 6, 2025 10:45
@konradkusiak97 konradkusiak97 force-pushed the Adde2eTestForCopyEngine branch from fff0709 to 9171ec0 Compare March 10, 2025 11:25
@konradkusiak97 konradkusiak97 force-pushed the Adde2eTestForCopyEngine branch from 9171ec0 to 835aca2 Compare March 10, 2025 19:08
@konradkusiak97 konradkusiak97 marked this pull request as ready for review March 11, 2025 09:33
@konradkusiak97 konradkusiak97 requested a review from a team as a code owner March 11, 2025 09:33
@konradkusiak97 konradkusiak97 requested a review from Bensuo March 11, 2025 09:33
Copy link
Contributor

@Bensuo Bensuo left a comment

Choose a reason for hiding this comment

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

LGTM

@konradkusiak97
Copy link
Contributor Author

@intel/llvm-gatekeepers please merge.

@sommerlukas sommerlukas merged commit 4fa727a into intel:sycl Mar 11, 2025
23 checks passed
adamfidel pushed a commit to reble/llvm that referenced this pull request Mar 11, 2025
…l#17324)

Following intel#16830, this PR adds an
additional e2e graph test to invoke copy engine on L0 devices and test
its behaviour.
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.

4 participants