Skip to content

[Driver] [SYCL] Fix SYCL compile time device lib link in tests. #22048

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

Open
srividya-sundaram wants to merge 3 commits into
base: sycl
Choose a base branch
from
+9 −5
Open

[Driver] [SYCL] Fix SYCL compile time device lib link in tests. #22048

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
0fe58e9
Fix SYCL compile time device lib link in tests.
srividya-sundaram May 19, 2026
e6f976a
[SYCL][test] Fix sycl-time-trace-actual compile+link test with -nostdlib
srividya-sundaram May 19, 2026
f5b94c6
Add runtime stubs.
srividya-sundaram May 19, 2026
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
14 changes: 9 additions & 5 deletions clang/test/Driver/sycl-time-trace-actual.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@

// Test 1: Compile-only mode with explicit trace directory
// RUN: %clang --target=x86_64-unknown-linux-gnu -fsycl --offload-new-driver \
// RUN: -c -ftime-trace=%t/traces -ftime-trace-granularity=0 \
// RUN: --sysroot=%S/Inputs/SYCL -c -ftime-trace=%t/traces -ftime-trace-granularity=0 \
// RUN: -ftime-trace-verbose %t/src/test.cpp -o %t/test.o

// Verify host and device trace files were generated
Expand All @@ -27,7 +27,7 @@
// When using -c with -o, trace files should be named based on the -o output
// RUN: rm -rf %t/traces2 && mkdir -p %t/traces2
// RUN: %clang --target=x86_64-unknown-linux-gnu -fsycl --offload-new-driver \
// RUN: -c -ftime-trace=%t/traces2 %t/src/test.cpp -o %t/different_name.o
// RUN: --sysroot=%S/Inputs/SYCL -c -ftime-trace=%t/traces2 %t/src/test.cpp -o %t/different_name.o

// Should be named after the -o output (different_name), not the source (test.cpp)
// RUN: ls %t/traces2/ | FileCheck %s --check-prefix=CHECK-TEST2
Expand All @@ -38,7 +38,7 @@
// In compile+link mode, trace files are named based on source file
// RUN: rm -rf %t/traces3 && mkdir -p %t/traces3
// RUN: %clang --target=x86_64-unknown-linux-gnu -fsycl --offload-new-driver \
// RUN: -ftime-trace=%t/traces3 %t/src/test.cpp -o %t/myapp
// RUN: --sysroot=%S/Inputs/SYCL -nostdlib -ftime-trace=%t/traces3 %t/src/test.cpp -o %t/myapp

// Frontend traces should still be generated (named after source file)
// RUN: ls %t/traces3/ | FileCheck %s --check-prefix=CHECK-TEST3
Expand All @@ -50,7 +50,7 @@
// RUN: cp %s %t/src2/file1.cpp
// RUN: cp %s %t/src2/file2.cpp
// RUN: %clang --target=x86_64-unknown-linux-gnu -fsycl --offload-new-driver \
// RUN: -c -ftime-trace=%t/traces4 %t/src2/file1.cpp %t/src2/file2.cpp
// RUN: --sysroot=%S/Inputs/SYCL -c -ftime-trace=%t/traces4 %t/src2/file1.cpp %t/src2/file2.cpp

// Should have clean trace file names since basenames are unique
// RUN: ls %t/traces4/ | FileCheck %s --check-prefix=CHECK-UNIQUE
Expand All @@ -65,7 +65,7 @@
// RUN: cp %s %t/dir1/test.cpp
// RUN: cp %s %t/dir2/test.cpp
// RUN: %clang --target=x86_64-unknown-linux-gnu -fsycl --offload-new-driver \
// RUN: -c -ftime-trace=%t/traces5 %t/dir1/test.cpp %t/dir2/test.cpp
// RUN: --sysroot=%S/Inputs/SYCL -c -ftime-trace=%t/traces5 %t/dir1/test.cpp %t/dir2/test.cpp

// Should have 4 trace files (2 sources × 2 traces each)
// Verify they use directory names to disambiguate (dir1-test, dir2-test)
Expand All @@ -76,6 +76,10 @@
// CHECK-COLLISION-DAG: dir2-test-sycl-spir64-unknown-unknown.json

// Minimal SYCL code for testing
// Provide stub runtime registration hooks so link mode works in this
// driver-only test environment without pulling in the SYCL runtime.
extern "C" void __sycl_register_lib(void *) {}
extern "C" void __sycl_unregister_lib(void *) {}
int main() {
return 0;
}
Loading