[PATCH] Add LLVM_REQUIRES_EH and LLVM_REQUIRES_RTTI as options in CMake

[Brad King]

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
set.

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.

-Brad