[PATCH] D30997: CMake: Add LLVM_DYLIB_SYMBOL_VERSIONING option

Evangelos Foutras via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 10 19:35:32 PDT 2018


foutrelis added a comment.

In https://reviews.llvm.org/D30997#1095549, @tstellar wrote:

> I think this option should be dropped.  When I added it, I was unaware of  https://reviews.llvm.org/D31524, which was being developed at the same time.


Thanks for clarifying that it's not needed; I thought I could be missing some use case for it (e.g. platforms other than Linux, BSD).

Removing it would be great so as to avoid confusion about it (and possibly libLLVM.so symbols being versioned differently than LLVM_x.y if the option is enabled).


Repository:
  rL LLVM

https://reviews.llvm.org/D30997





More information about the llvm-commits mailing list