Skip to content

Commit 57d908c

Browse files
committed
[SYCL][Graph] Reduce work in update tests
On certain setups these tests take a long time to execute, impairing the ability to run the whole suite of graph E2E tests in lit. Reduce the number of graph execution iterations used in tests to alleviate this situation. There is an issue with free-function kernels taking longer that the non-free function equivalent, documented in CMPLRLLVM-64841, so only use a single execution iteration for now.
1 parent 44c58bb commit 57d908c

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

sycl/test-e2e/Graph/Update/FreeFunctionKernels/update_with_indices_ordering.cpp

+6-2
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,13 @@ int main() {
2020

2121
// Use a large N to try and make the kernel slow
2222
const size_t N = 1 << 16;
23+
24+
// Reduce amount of work compared to version of test without free functions
25+
// due to CMPLRLLVM-64841
26+
//
2327
// Loop inside kernel to make even slower (too large N runs out of memory)
24-
const size_t NumKernelLoops = 4;
25-
const size_t NumSubmitLoops = 8;
28+
const size_t NumKernelLoops = 2;
29+
const size_t NumSubmitLoops = 1;
2630

2731
exp_ext::command_graph Graph{Ctxt, Queue.get_device()};
2832

sycl/test-e2e/Graph/Update/update_with_indices_ordering.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ int main() {
1818
const size_t N = 1 << 16;
1919
// Loop inside kernel to make even slower (too large N runs out of memory)
2020
const size_t NumKernelLoops = 4;
21-
const size_t NumSubmitLoops = 8;
21+
const size_t NumSubmitLoops = 2;
2222

2323
exp_ext::command_graph Graph{Queue.get_context(), Queue.get_device()};
2424

0 commit comments

Comments
 (0)