Skip to content

Commit 39e96b2

Browse files
AD2605Rbiessy
andauthored
Apply suggestions from code review
Co-authored-by: Romain Biessy <[email protected]>
1 parent 1371952 commit 39e96b2

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

ggml/src/ggml-sycl/CMakeLists.txt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -52,13 +52,13 @@ target_compile_options(ggml-sycl PRIVATE "-Wno-narrowing")
5252
set(GGML_SYCL_DNNL 0)
5353
if(GGML_SYCL_DNN)
5454

55-
# The nightly dpcpp compiler recently stopped shipping libOpenCL
55+
# The nightly dpcpp compiler may not ship libOpenCL
5656
# When using a nightly build of oneDNN built with the said nightly compiler, it links against the target OpenCL::OpenCL
57-
# For context,this line in the oneDNN CMake:
57+
# For context, this line in the oneDNN CMake:
5858
# https://github.com/uxlfoundation/oneDNN/blob/743e4273d18bcb0e733723bf9289f14e8366687f/cmake/SYCL.cmake#L127
5959
# and when using this build of oneDNN in llama, it would expect the target OpenCL::OpenCL to be available.
6060
# Hence the below find_package call is required. At the time of writing, the release compiler
61-
# ships libOpenCL,hence only finding the package when the compiler is clang (nightly).
61+
# ships libOpenCL, hence only finding the package when the compiler is clang (nightly).
6262

6363
if (CMAKE_CXX_COMPILER_ID STREQUAL "Clang")
6464
find_package(OpenCL REQUIRED)

0 commit comments

Comments
 (0)