[llvm-commits] [cfe-commits] [Review request] CMake: Add the new option "LLVM_LIT_ARGS"

Douglas Gregor dgregor at apple.com
Mon Nov 8 18:41:30 PST 2010


Looks good to me, unless Óscar objects.

	- Doug

On Nov 8, 2010, at 8:13 PM, NAKAMURA Takumi wrote:

> I propose adding, for CMake, the option "LLVM_LIT_ARGS".
> 
> By default LLVM_LIT_ARGS is set;
> 
> MSVC or XCODE: -sv --no-progress-bar
> others: -sv
> 
> *  0001-CMake-Add-the-new-option-LLVM_LIT_ARGS.patch
> 
>  for llvm tree.
> 
> *  0001-test-CMakeLists.txt-Use-LLVM_LIT_ARGS-and-remove-red.patch
> 
>  for clang tree. it depends on the former patch.
> 
> I wonder whether the name "LLVM_LIT_ARGS" would be suitable or not :)
> (would be better to split to new namespace LIT_**** ?)
> Please gimme any advices.
> 
> 
> Thank you, ...Takumi
> <0001-CMake-Add-the-new-option-LLVM_LIT_ARGS.patch><0001-test-CMakeLists.txt-Use-LLVM_LIT_ARGS-and-remove-red.patch>_______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits





More information about the llvm-commits mailing list