Skip to content

[SYCL] Fix KernelFusion/abort_fusion.cpp in "--param ze_debug=-1" mode #9077

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.

Sign up for GitHub

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 2 commits into from
Apr 18, 2023
Merged

[SYCL] Fix KernelFusion/abort_fusion.cpp in "--param ze_debug=-1" mode #9077

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
45c52b9
[SYCL] Fix KernelFusion/abort_fusion.cpp in "--param ze_debug=-1" mode
aelovikov-intel Apr 14, 2023
9cb2bbb
Address review comments
aelovikov-intel Apr 17, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions sycl/test-e2e/KernelFusion/abort_fusion.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,8 @@ void performFusion(queue &q, range<Kernel1Dim> k1Global,
} else {
std::cout << "COMPUTATION OK\n";
}

assert(numErrors == 0);
}

int main() {
Expand All @@ -84,9 +86,9 @@ int main() {
// fusion being aborted.
performFusion<class Kernel1_3, class Kernel2_3>(q, range<1>{dataSize},
range<1>{16});
// CHECK: ERROR: JIT compilation for kernel fusion failed with message:
// CHECK: ERROR: JIT compilation for kernel fusion failed with message:
// CHECK-NEXT: Cannot fuse kernels with different offsets or local sizes
// CHECK-NEXT: COMPUTATION OK
// CHECK: COMPUTATION OK
Copy link
Contributor

Choose a reason for hiding this comment

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

Do I assume correctly that this change is motivated by additional output with ze_debug in between the output expected by the test?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes.


return 0;
}