[libclc] [libclc][CMake] Rename opencl to clc in add_libclc_library, update comment (PR #186544)
Wenju He via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 13 18:10:46 PDT 2026
https://github.com/wenju-he updated https://github.com/llvm/llvm-project/pull/186544
>From ac5386cc9b255b668c53309aaf98c05cd466609a Mon Sep 17 00:00:00 2001
From: Wenju He <wenju.he at intel.com>
Date: Sat, 14 Mar 2026 02:00:00 +0100
Subject: [PATCH 1/2] [libclc][CMake] Rename opencl to clc in
add_libclc_library, update comment
Align with cmake function name.
---
libclc/cmake/modules/AddLibclc.cmake | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/libclc/cmake/modules/AddLibclc.cmake b/libclc/cmake/modules/AddLibclc.cmake
index 479a68865ad30..4d4a19c2567c0 100644
--- a/libclc/cmake/modules/AddLibclc.cmake
+++ b/libclc/cmake/modules/AddLibclc.cmake
@@ -17,7 +17,7 @@ macro(libclc_configure_source_options path option)
)
endmacro()
-# Merges OpenCL C source file lists with priority deduplication.
+# Merges CL source file lists with priority deduplication.
#
# All arguments after the output variable name are treated as source file
# paths. When multiple files share the same basename, the last occurrence
@@ -146,9 +146,9 @@ function(link_libclc_builtin_library target_name)
)
endfunction()
-# Builds an OpenCL builtins library from sources, links it with any
-# internalized dependencies via link_libclc_builtin_library, and adds
-# a verification test for unresolved symbols.
+# Builds a builtins library from sources, links it with any internalized
+# dependencies via link_libclc_builtin_library, and adds a verification test
+# for unresolved symbols.
function(add_libclc_library target_name)
cmake_parse_arguments(ARG
""
@@ -167,19 +167,19 @@ function(add_libclc_library target_name)
message(FATAL_ERROR "SOURCES is required for add_libclc_library")
endif()
- set(opencl_lib ${target_name}_opencl_builtins)
- add_libclc_builtin_library(${opencl_lib}
+ set(clc_lib ${target_name}_clc_builtins)
+ add_libclc_builtin_library(${clc_lib}
SOURCES ${ARG_SOURCES}
COMPILE_OPTIONS ${ARG_COMPILE_OPTIONS}
INCLUDE_DIRS ${ARG_INCLUDE_DIRS}
COMPILE_DEFINITIONS ${ARG_COMPILE_DEFINITIONS}
- FOLDER "libclc/Device IR/OpenCL"
+ FOLDER "libclc/Device IR/Intermediate"
)
link_libclc_builtin_library(${target_name}
ARCH ${ARG_ARCH}
TRIPLE ${ARG_TRIPLE}
- LIBRARIES ${opencl_lib}
+ LIBRARIES ${clc_lib}
INTERNALIZE_LIBRARIES ${ARG_INTERNALIZE_LIBRARIES}
OPT_FLAGS ${ARG_OPT_FLAGS}
OUTPUT_FILENAME "${ARG_OUTPUT_FILENAME}"
>From ada2aa6953a85da59c194a7794cd4bf05e55eb24 Mon Sep 17 00:00:00 2001
From: Wenju He <wenju.he at intel.com>
Date: Sat, 14 Mar 2026 02:10:10 +0100
Subject: [PATCH 2/2] restore `OpenCL C` comment
---
libclc/cmake/modules/AddLibclc.cmake | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libclc/cmake/modules/AddLibclc.cmake b/libclc/cmake/modules/AddLibclc.cmake
index 4d4a19c2567c0..4793e92d2970e 100644
--- a/libclc/cmake/modules/AddLibclc.cmake
+++ b/libclc/cmake/modules/AddLibclc.cmake
@@ -17,7 +17,7 @@ macro(libclc_configure_source_options path option)
)
endmacro()
-# Merges CL source file lists with priority deduplication.
+# Merges OpenCL C source file lists with priority deduplication.
#
# All arguments after the output variable name are treated as source file
# paths. When multiple files share the same basename, the last occurrence
More information about the cfe-commits
mailing list