r219994 - Appease the buildbots with the special case for non-set variables

Justin Bogner mail at justinbogner.com
Fri Oct 17 15:07:45 PDT 2014


Filipe Cabecinhas <me at filcab.net> writes:
> Author: filcab
> Date: Thu Oct 16 18:54:26 2014
> New Revision: 219994
>
> URL: http://llvm.org/viewvc/llvm-project?rev=219994&view=rev
> Log:
> Appease the buildbots with the special case for non-set variables
>
> Modified:
>     cfe/trunk/test/lit.cfg
>
> Modified: cfe/trunk/test/lit.cfg
> URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/lit.cfg?rev=219994&r1=219993&r2=219994&view=diff
> ==============================================================================
> --- cfe/trunk/test/lit.cfg (original)
> +++ cfe/trunk/test/lit.cfg Thu Oct 16 18:54:26 2014
> @@ -271,8 +271,9 @@ config.substitutions.append( ('%clangxx'
>  config.substitutions.append( ('%clang', ' ' + config.clang + ' ') )
>  config.substitutions.append( ('%test_debuginfo', ' ' + config.llvm_src_root + '/utils/test_debuginfo.pl ') )
>  config.substitutions.append( ('%itanium_abi_triple', makeItaniumABITriple(config.target_triple)) )
> -config.substitutions.append( ('%itanium_abi_host_triple', makeItaniumABITriple(config.host_triple)) )
>  config.substitutions.append( ('%ms_abi_triple', makeMSABITriple(config.target_triple)) )
> +if config.host_triple != '@LLVM_HOST_TRIPLE@':
> +  config.substitutions.append( ('%itanium_abi_host_triple', makeItaniumABITriple(config.host_triple)) )

Should this set %itanium_abi_host_triple to config.target_triple in the
case where config.host_triple isn't set? Otherwise the substitution
won't exist at all.

>  # FIXME: Find nicer way to prohibit this.
>  config.substitutions.append(
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits



More information about the cfe-commits mailing list