Skip to content

Conversation

omarahmed1111
Copy link
Contributor

@omarahmed1111 omarahmed1111 commented May 15, 2025

Currently L0/opencl benchmarks is still built on computeBench even when we specify that we need to benchmark sycl on top of cuda that creates an error sometimes when the execution environment doesn't have L0 setup. That PR will close building L0/opencl benchmarks on computeBench when we specify cuda target for benchmarking.

@omarahmed1111 omarahmed1111 force-pushed the switch-off-l0-for-compute-bench-cuda branch from c3118e7 to 4b26bcf Compare May 15, 2025 14:07
@omarahmed1111 omarahmed1111 marked this pull request as ready for review May 19, 2025 09:43
@omarahmed1111 omarahmed1111 requested a review from a team as a code owner May 19, 2025 09:43
@omarahmed1111
Copy link
Contributor Author

omarahmed1111 commented May 19, 2025

@intel/llvm-gatekeepers please merge, the changes here are not related to the ci failures. Thanks!

@omarahmed1111 omarahmed1111 force-pushed the switch-off-l0-for-compute-bench-cuda branch from 4b26bcf to b9c6f88 Compare May 19, 2025 11:16
@aelovikov-intel
Copy link
Contributor

@sarnex , this is fine, right?

@omarahmed1111 omarahmed1111 force-pushed the switch-off-l0-for-compute-bench-cuda branch from b9c6f88 to d4f4da1 Compare May 20, 2025 11:07
@omarahmed1111 omarahmed1111 force-pushed the switch-off-l0-for-compute-bench-cuda branch from d4f4da1 to 4979c9f Compare May 20, 2025 11:12
@omarahmed1111 omarahmed1111 changed the title Switch off L0 for computeBench on SYCL for CUDA [Benchmark] Switch off L0 for computeBench on SYCL for CUDA May 20, 2025
@omarahmed1111 omarahmed1111 force-pushed the switch-off-l0-for-compute-bench-cuda branch from 4979c9f to 8ce26bb Compare May 21, 2025 09:26
@omarahmed1111
Copy link
Contributor Author

@intel/llvm-gatekeepers Gentle ping to merge this when available, the failure here is unrelated to the PR, it falls in another PRs with the same failure like here

@sarnex sarnex merged commit e7ab07d into intel:sycl May 21, 2025
24 of 28 checks passed
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.

5 participants