[PATCH] D14040: [CMake] make name of libLLVM configurable

Andrew Wilkins via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 27 08:00:44 PDT 2015


axw added a comment.

> Not sure I like the "." idea... This is not obvious. What is wrong with unset ?


I agree it's not very obvious, but it remains that we need some way of preventing SOVERSION from being set.

Can you please explain what you mean by "unset"? Are you suggesting "unset" as an alternative string to compare against, or are you referring to the CMake unset() command? If you mean the command, I'm not sure it would help.

> By the way, I think we also update the documentation.


Sure, I'll take a pass over the docs to see what needs updating.


http://reviews.llvm.org/D14040





More information about the llvm-commits mailing list