Skip to content

Commit 4f560f8

Browse files
committed
update per review comments
1 parent 7bfffff commit 4f560f8

File tree

3 files changed

+7
-16
lines changed

3 files changed

+7
-16
lines changed

clang/lib/Driver/ToolChains/SYCL.cpp

Lines changed: 4 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -64,19 +64,10 @@ const char *SYCLInstallationDetector::findLibspirvPath(
6464

6565
const SmallString<64> Basename =
6666
getLibSpirvBasename(DeviceTriple, HostTriple);
67-
auto searchAt = [&](StringRef Path, const Twine &a = "", const Twine &b = "",
68-
const Twine &c = "") -> const char * {
69-
SmallString<128> LibraryPath(Path);
70-
llvm::sys::path::append(LibraryPath, a, b, c, Basename);
71-
72-
if (D.getVFS().exists(LibraryPath))
73-
return Args.MakeArgString(LibraryPath);
74-
75-
return nullptr;
76-
};
77-
78-
if (const char *R = searchAt(D.ResourceDir, "lib", "libclc"))
79-
return R;
67+
SmallString<256> LibclcPath(D.ResourceDir);
68+
llvm::sys::path::append(LibclcPath, "lib", "libclc", Basename);
69+
if (D.getVFS().exists(LibclcPath))
70+
return Args.MakeArgString(LibclcPath);
8071

8172
return nullptr;
8273
}

clang/test/Driver/Inputs/SYCL/bin/.gitkeep

Whitespace-only changes.

clang/test/Driver/sycl-libspirv-toolchain.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,10 @@
33
// DEFINE: %{resource_dir} = %/S/Inputs/SYCL/lib/clang/resource_dir
44

55
// RUN: %clang -### -resource-dir %{resource_dir} -fsycl -fsycl-targets=nvptx64-nvidia-cuda -nocudalib -target x86_64-unknown-windows-msvc %s 2>&1 \
6-
// RUN: | FileCheck %s -DRESOURCE_DIR=%{resource_dir} --check-prefixes=CHECK-WINDOWS
6+
// RUN: | FileCheck %s --check-prefixes=CHECK-WINDOWS
77
// RUN: %clang -### -resource-dir %{resource_dir} -fsycl -fsycl-targets=nvptx64-nvidia-cuda -nocudalib -target x86_64-unknown-windows-gnu %s 2>&1 \
8-
// RUN: | FileCheck %s -DRESOURCE_DIR=%{resource_dir} --check-prefixes=CHECK-WINDOWS
9-
// CHECK-WINDOWS: "-cc1"{{.*}} "-fsycl-is-device"{{.*}} "-mlink-builtin-bitcode" "[[RESOURCE_DIR]]{{.*[\\/]}}remangled-l32-signed_char.libspirv-nvptx64-nvidia-cuda.bc"
8+
// RUN: | FileCheck %s --check-prefixes=CHECK-WINDOWS
9+
// CHECK-WINDOWS: "-cc1"{{.*}} "-fsycl-is-device"{{.*}} "-mlink-builtin-bitcode" "{{.*[\\/]}}remangled-l32-signed_char.libspirv-nvptx64-nvidia-cuda.bc"
1010
//
1111
// RUN: %clang -### -resource-dir %{resource_dir} -fsycl -fsycl-targets=nvptx64-nvidia-cuda -nocudalib -target x86_64-unknown-linux-gnu %s 2>&1 \
1212
// RUN: | FileCheck %s -DRESOURCE_DIR=%{resource_dir} --check-prefixes=CHECK-LINUX

0 commit comments

Comments
 (0)