Skip to content

Commit

Permalink
[SYCL][Graph][E2E] Reduce work in intensive update tests running (#16632
Browse files Browse the repository at this point in the history
)

On certain setups these two 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.
  • Loading branch information
EwanC authored Jan 15, 2025
1 parent 5bf9731 commit d4e3e45
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,11 @@ int main() {

// Use a large N to try and make the kernel slow
const size_t N = 1 << 16;
// Loop inside kernel to make even slower (too large N runs out of memory)
const size_t NumKernelLoops = 4;
const size_t NumSubmitLoops = 8;

// Reduce amount of work compared to version of test without free functions
// due to CMPLRLLVM-64841
const size_t NumKernelLoops = 1;
const size_t NumSubmitLoops = 1;

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

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ int main() {
const size_t N = 1 << 16;
// Loop inside kernel to make even slower (too large N runs out of memory)
const size_t NumKernelLoops = 4;
const size_t NumSubmitLoops = 8;
const size_t NumSubmitLoops = 2;

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

Expand Down

0 comments on commit d4e3e45

Please sign in to comment.