[PATCH] Add LLVM_REQUIRES_EH and LLVM_REQUIRES_RTTI as options in CMake
brad.king at kitware.com
Mon Jul 21 06:13:14 PDT 2014
On 07/20/2014 01:57 PM, Dan Liew wrote:
> An alternative would be create options LLVM_ENABLE_RTTI and
> LLVM_ENABLE_EH (this naming would be more similar to existing LLVM
> option names) and reserve LLVM_REQUIRES_EH and LLVM_REQUIRES_RTTI for
> internal use.
This makes more sense to me. The LLVM_ENABLE_* options would provide
defaults for the LLVM_REQUIRES_* variables when they are not otherwise
Is the intention here to follow up by exporting LLVM_REQUIRES_*
in LLVMConfig.cmake? That would give projects enough information
to know how to compile in a way that is compatible with the LLVM
libraries. IMO that is better than exporting a _FLAGS variable
that is compiler-specific.
More information about the llvm-commits