[PATCH] D31524: Add a linker script to version LLVM symbols

[Sylvestre Ledru via Phabricator via llvm-commits]

sylvestre.ledru added a comment.

> Can you elaborate on what you mean by this? Do you mean LLVM_5.0 doesn't depend on LLVM_4.0?

Yeah, sorry. I will fix that in the commit



================
Comment at: cmake/modules/AddLLVM.cmake:85
     add_custom_command(OUTPUT ${native_export_file}
-      COMMAND echo "{" > ${native_export_file}
+      COMMAND echo "LLVM_${LLVM_VERSION_MAJOR}.${LLVM_VERSION_MINOR} {" > ${native_export_file}
       COMMAND grep -q "[[:alnum:]]" ${export_file} && echo "  global:" >> ${native_export_file} || :
----------------
rnk wrote:
> sylvestre.ledru wrote:
> > rnk wrote:
> > > Is this correct for DSOs that aren't libLLVM.so.N.M? This function is called from LLDB and elsewhere.
> > It doesn't have too. libLLVM.so.N.M is for the SONAME.
> > However, I can rename it, no worries
> So, this doesn't refer to LLVM the DSO, it refers to LLVM the project. After this change, libclang.so will have symbols with LLVM_N.M as their version, right?
Yes, I can rename it to LLVM_TOOLCHAIN_N.M if you prefer but this is a bit long.



Repository:
  rL LLVM

https://reviews.llvm.org/D31524