Skip to content

[libclang][Cygwin] Provide unversioned DLL file alongside versioned one #147132

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
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

[libclang][Cygwin] Provide unversioned DLL file alongside versioned one #147132

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
9198e98
[libclang][Cygwin] Put unversioned DLL file aside of versioned DLL file
kikairoya Jun 28, 2025
64e727a
rewrite comment
kikairoya Jul 5, 2025
15003b1
use a symlink, don't copy
kikairoya Jul 6, 2025
f28f49f
link to relative path
kikairoya Jul 7, 2025
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
23 changes: 18 additions & 5 deletions clang/tools/libclang/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -157,15 +157,28 @@ if(ENABLE_STATIC)
endif()

if(ENABLE_SHARED)
if(WIN32)
if(WIN32 OR CYGWIN)
set_target_properties(libclang
PROPERTIES
VERSION ${LIBCLANG_LIBRARY_VERSION}
DEFINE_SYMBOL _CINDEX_LIB_)
# Avoid declaring clang c++ symbols that are statically linked into libclang as dllimport'ed.
# If llvm/libclang-cpp dll is also being built for windows clang c++ symbols will still be
# implicitly be exported from libclang.
target_compile_definitions(libclang PRIVATE CLANG_BUILD_STATIC)
if (CYGWIN)
# On Cygwin, a shared library target generates "cyg${OUTPUT_NAME}-${VERSION}.dll"
# and "lib${OUTPUT_NAME}.dll.a", but does not generate an unversioned symlink
# "cyg${OUTPUT_NAME}.dll" as is common on other Unix platforms.
# However, tools like clang-python or direct dlopen() calls usually expect the
# unversioned form (e.g. "cygclang.dll").
# Therefore, we create a symlink named without the version suffix to
# improve compatibility.
set(UNPAINTED_TARGET_NAME "$<TARGET_FILE_DIR:libclang>/$<TARGET_FILE_PREFIX:libclang>clang$<TARGET_FILE_SUFFIX:libclang>")
add_custom_command(TARGET libclang POST_BUILD
COMMAND ${CMAKE_COMMAND} -E create_symlink "$<TARGET_FILE_NAME:libclang>" "${UNPAINTED_TARGET_NAME}")
install(FILES "${UNPAINTED_TARGET_NAME}" DESTINATION "${CMAKE_INSTALL_BINDIR}" COMPONENT libclang)
endif()
# Avoid declaring clang c++ symbols that are statically linked into libclang as dllimport'ed.
# If llvm/libclang-cpp dll is also being built for windows clang c++ symbols will still be
# implicitly be exported from libclang.
target_compile_definitions(libclang PRIVATE CLANG_BUILD_STATIC)
elseif(APPLE)
set(LIBCLANG_LINK_FLAGS " -Wl,-compatibility_version -Wl,1")
set(LIBCLANG_LINK_FLAGS "${LIBCLANG_LINK_FLAGS} -Wl,-current_version -Wl,${LLVM_VERSION_MAJOR}.${LLVM_VERSION_MINOR}.${LLVM_VERSION_PATCH}")
Expand Down