[clang] [llvm] [CMake] Version Darwin dylib identities (PR #189004)

[Tamir Duberstein via llvm-commits]

================
@@ -170,11 +170,12 @@ if(ENABLE_SHARED)
       # 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}")
-
-    set_property(TARGET libclang APPEND_STRING PROPERTY
-                 LINK_FLAGS ${LIBCLANG_LINK_FLAGS})
+    if(LLVM_VERSIONED_DYLIB_NAME_ON_DARWIN)
----------------
tamird wrote:

Yes, this PR preserves that. `MACHO_COMPATIBILITY_VERSION` and `MACHO_CURRENT_VERSION` take precedence over `VERSION`.

https://github.com/llvm/llvm-project/pull/189004