Skip to content

Commit

Permalink
[SYCL][E2E] Use %threads_lib expansion instead of custom expansion …
Browse files Browse the repository at this point in the history
…in Graph test (#15370)

Removes custom expansion only used in this test, in favor of using
%threads_lib. The custom expansion used `-pthread` explicitly, which
produced an unknown argument warning when running using the clang-cl
driver.
  • Loading branch information
ayylol committed Sep 16, 2024
1 parent 99d4cc1 commit 371ef5e
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 6 deletions.
2 changes: 1 addition & 1 deletion sycl/test-e2e/Graph/Threading/submit.cpp
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// RUN: %{build_pthread_inc} -o %t.out
// RUN: %{build} %threads_lib -o %t.out
// RUN: %{run} %t.out
// RUN: %if level_zero %{env SYCL_PI_LEVEL_ZERO_USE_IMMEDIATE_COMMANDLISTS=0 %{l0_leak_check} %{run} %t.out 2>&1 | FileCheck %s --implicit-check-not=LEAK %}
// Extra run to check for immediate-command-list in Level Zero
Expand Down
5 changes: 0 additions & 5 deletions sycl/test-e2e/format.py
Original file line number Diff line number Diff line change
Expand Up @@ -186,11 +186,6 @@ def execute(self, test, litConfig):
else:
substitutions.append(("%{l0_leak_check}", "env UR_L0_LEAKS_DEBUG=1"))

compilation_cmd_pthread = (
"%clangxx -pthread -fsycl -fsycl-targets=%{sycl_triple} %s"
)
substitutions.append(("%{build_pthread_inc}", compilation_cmd_pthread))

def get_extra_env(sycl_devices):
# Note: It's possible that the system has a device from below but
# current llvm-lit invocation isn't configured to include it. We
Expand Down

0 comments on commit 371ef5e

Please sign in to comment.