serge-sans-paille added inline comments.
================
Comment at: clang/tools/clang-shlib/CMakeLists.txt:2
+# In the main branch, LLVM_ABI_REVISION should always be 0. In the release
+# branches, this should be incremented before each release candidate every
+# time the ABI of libclang-cpp.so changes.
----------------
This comment is a bit misleading: should it be for each RC *or* for each RC
that implies an ABI change? And what about the (very rare) minor version
release, like 11.1.0? My understanding is that they implie an SONAME bump,
maybe that should be stated somewhere?
I think it would be great to have that ABI policy documented somewhere in the
official doc, and not only as a cmake comment, as every project linking with
clang or llvm is impacted.
================
Comment at: llvm/cmake/modules/AddLLVM.cmake:595
# Since 4.0.0, the ABI version is indicated by the major version
- SOVERSION ${LLVM_VERSION_MAJOR}${LLVM_VERSION_SUFFIX}
- VERSION ${LLVM_VERSION_MAJOR}${LLVM_VERSION_SUFFIX})
+ SOVERSION ${LLVM_VERSION_MAJOR}.${LLVM_ABI_REVISION}
+ VERSION ${LLVM_VERSION_MAJOR}.${LLVM_ABI_REVISION})
----------------
Nit: strange indent.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105594/new/
https://reviews.llvm.org/D105594
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
