[llvm-branch-commits] [clang] 8876f16 - Revert "Fix a cmake error when using the Xcode generator. (#119403)"

via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Fri Jan 10 05:48:37 PST 2025


Author: Aaron Ballman
Date: 2025-01-10T08:48:34-05:00
New Revision: 8876f169829dd3c8dfebcfe9652fb1059ce75325

URL: https://github.com/llvm/llvm-project/commit/8876f169829dd3c8dfebcfe9652fb1059ce75325
DIFF: https://github.com/llvm/llvm-project/commit/8876f169829dd3c8dfebcfe9652fb1059ce75325.diff

LOG: Revert "Fix a cmake error when using the Xcode generator. (#119403)"

This reverts commit 1842a3d833d934793012c717e98b10d51193fd0d.

Added: 
    

Modified: 
    clang/cmake/modules/AddClang.cmake

Removed: 
    


################################################################################
diff  --git a/clang/cmake/modules/AddClang.cmake b/clang/cmake/modules/AddClang.cmake
index cdc8bd5cd503b4..091aec98e93ca3 100644
--- a/clang/cmake/modules/AddClang.cmake
+++ b/clang/cmake/modules/AddClang.cmake
@@ -109,14 +109,13 @@ macro(add_clang_library name)
   llvm_add_library(${name} ${LIBTYPE} ${ARG_UNPARSED_ARGUMENTS} ${srcs})
 
   if(MSVC AND NOT CLANG_LINK_CLANG_DYLIB)
-    # Make sure all consumers also turn off visibility macros so they're not
-    # trying to dllimport symbols.
+    # Make sure all consumers also turn off visibility macros so there not trying to dllimport symbols.
     target_compile_definitions(${name} PUBLIC CLANG_BUILD_STATIC)
     if(TARGET "obj.${name}")
       target_compile_definitions("obj.${name}" PUBLIC CLANG_BUILD_STATIC)
     endif()
-  elseif(TARGET "obj.${name}" AND NOT ARG_SHARED AND NOT ARG_STATIC)
-    # Clang component libraries linked to clang-cpp are declared without SHARED or STATIC
+  elseif(NOT ARG_SHARED AND NOT ARG_STATIC)
+    # Clang component libraries linked in to clang-cpp are declared without SHARED or STATIC
     target_compile_definitions("obj.${name}" PUBLIC CLANG_EXPORTS)
   endif()
 


        


More information about the llvm-branch-commits mailing list