[llvm] r280013 - [CMake] Make LLVMConfig.cmake variable names match in-tree names

Brad King via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 31 07:31:50 PDT 2016

On 08/29/2016 05:26 PM, Chris Bieneman via llvm-commits wrote:
> New Revision: 280013
> URL: http://llvm.org/viewvc/llvm-project?rev=280013&view=rev
> This renames:
> The versions ending in S are not used in-tree anywhere. This also
> cleans up LLVM_LIBRARY_DIR being set to the same value with and
> without the "S".

Please don't do this.  It breaks compatibility with existing consumers
of LLVM that build with CMake.  Also it breaks the convention documented
by CMake for public-facing package information variables:


The public-facing variables should be plural-named because they may have
zero, one, or more than one entries in a list.  Even if they happen to
have only one now they may have more in the future.


More information about the llvm-commits mailing list