diff options
author | Bruno Cardoso Lopes <bruno.cardoso@gmail.com> | 2017-10-13 17:11:13 +0000 |
---|---|---|
committer | Bruno Cardoso Lopes <bruno.cardoso@gmail.com> | 2017-10-13 17:11:13 +0000 |
commit | 72f20cba6a67fdf977709fbece95d1b1b4c53322 (patch) | |
tree | a4028c5eb897c0df3ca3e87878689a2bdcbab8eb /test/lit.cfg.py | |
parent | 8ebf4572cb0d7d66ad53b85ef8f53483b474fcbb (diff) |
Revert "[lit] Raise the logic for enabling clang & lld substitutions to llvm."
This reverts commit r315627, fixing bot failures:
http://green.lab.llvm.org/green/job/clang-stage1-configure-RA
LIT is failing to properly apply substitution to debuginfo-tests
after this change.
rdar://problem/34979568
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@315712 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test/lit.cfg.py')
-rw-r--r-- | test/lit.cfg.py | 155 |
1 files changed, 136 insertions, 19 deletions
diff --git a/test/lit.cfg.py b/test/lit.cfg.py index d2f0491821..a6f9bb3d86 100644 --- a/test/lit.cfg.py +++ b/test/lit.cfg.py @@ -39,33 +39,71 @@ config.test_source_root = os.path.dirname(__file__) # test_exec_root: The root path where tests should be run. config.test_exec_root = os.path.join(config.clang_obj_root, 'test') -llvm_config.use_default_substitutions() - -llvm_config.use_clang() +# Clear some environment variables that might affect Clang. +# +# This first set of vars are read by Clang, but shouldn't affect tests +# that aren't specifically looking for these features, or are required +# simply to run the tests at all. +# +# FIXME: Should we have a tool that enforces this? + +# safe_env_vars = ('TMPDIR', 'TEMP', 'TMP', 'USERPROFILE', 'PWD', +# 'MACOSX_DEPLOYMENT_TARGET', 'IPHONEOS_DEPLOYMENT_TARGET', +# 'VCINSTALLDIR', 'VC100COMNTOOLS', 'VC90COMNTOOLS', +# 'VC80COMNTOOLS') +possibly_dangerous_env_vars = ['COMPILER_PATH', 'RC_DEBUG_OPTIONS', + 'CINDEXTEST_PREAMBLE_FILE', 'LIBRARY_PATH', + 'CPATH', 'C_INCLUDE_PATH', 'CPLUS_INCLUDE_PATH', + 'OBJC_INCLUDE_PATH', 'OBJCPLUS_INCLUDE_PATH', + 'LIBCLANG_TIMING', 'LIBCLANG_OBJTRACKING', + 'LIBCLANG_LOGGING', 'LIBCLANG_BGPRIO_INDEX', + 'LIBCLANG_BGPRIO_EDIT', 'LIBCLANG_NOTHREADS', + 'LIBCLANG_RESOURCE_USAGE', + 'LIBCLANG_CODE_COMPLETION_LOGGING'] +# Clang/Win32 may refer to %INCLUDE%. vsvarsall.bat sets it. +if platform.system() != 'Windows': + possibly_dangerous_env_vars.append('INCLUDE') + +llvm_config.clear_environment(possibly_dangerous_env_vars) + +# Tweak the PATH to include the tools dir and the scripts dir. +llvm_config.with_environment( + 'PATH', [config.llvm_tools_dir, config.clang_tools_dir], append_path=True) + +llvm_config.with_environment('LD_LIBRARY_PATH', [ + config.llvm_shlib_dir, config.llvm_libs_dir], append_path=True) # Propagate path to symbolizer for ASan/MSan. llvm_config.with_system_environment( ['ASAN_SYMBOLIZER_PATH', 'MSAN_SYMBOLIZER_PATH']) -config.substitutions.append(('%PATH%', config.environment['PATH'])) +llvm_config.use_default_substitutions() +# Discover the 'clang' and 'clangcc' to use. -# For each occurrence of a clang tool name, replace it with the full path to -# the build directory holding that tool. We explicitly specify the directories -# to search to ensure that we get the tools just built and not some random -# tools that might happen to be in the user's PATH. -tool_dirs = [config.clang_tools_dir, config.llvm_tools_dir] -tools = [ - 'c-index-test', 'clang-check', 'clang-diff', 'clang-format', 'opt', - ToolSubst('%clang_func_map', command=FindTool( - 'clang-func-mapping'), unresolved='ignore'), -] +def inferClang(PATH): + # Determine which clang to use. + clang = os.getenv('CLANG') -if config.clang_examples: - tools.append('clang-interpreter') + # If the user set clang in the environment, definitely use that and don't + # try to validate. + if clang: + return clang -llvm_config.add_tool_substitutions(tools, tool_dirs) + # Otherwise look in the path. + clang = lit.util.which('clang', PATH) + + if not clang: + lit_config.fatal("couldn't find 'clang' program, try setting " + 'CLANG in your environment') + + return clang + + +config.clang = inferClang(config.environment['PATH']).replace('\\', '/') +if not lit_config.quiet: + lit_config.note('using clang: %r' % config.clang) # Plugins (loadable modules) # TODO: This should be supplied by Makefile or autoconf. @@ -77,6 +115,87 @@ else: if has_plugins and config.llvm_plugin_ext: config.available_features.add('plugins') +config.substitutions.append(('%llvmshlibdir', config.llvm_shlib_dir)) +config.substitutions.append(('%pluginext', config.llvm_plugin_ext)) +config.substitutions.append(('%PATH%', config.environment['PATH'])) + +if config.clang_examples: + config.available_features.add('examples') + +builtin_include_dir = llvm_config.get_clang_builtin_include_dir(config.clang) + +tools = [ + # By specifying %clang_cc1 as part of the substitution, this substitution + # relies on repeated substitution, so must come before %clang_cc1. + ToolSubst('%clang_analyze_cc1', command='%clang_cc1', + extra_args=['-analyze', '%analyze']), + ToolSubst('%clang_cc1', command=config.clang, extra_args=[ + '-cc1', '-internal-isystem', builtin_include_dir, '-nostdsysteminc']), + ToolSubst('%clang_cpp', command=config.clang, + extra_args=['--driver-mode=cpp']), + ToolSubst('%clang_cl', command=config.clang, + extra_args=['--driver-mode=cl']), + ToolSubst('%clangxx', command=config.clang, + extra_args=['--driver-mode=g++']), + ToolSubst('%clang_func_map', command=FindTool( + 'clang-func-mapping'), unresolved='ignore'), + ToolSubst('%clang', command=config.clang), + ToolSubst('%test_debuginfo', command=os.path.join( + config.llvm_src_root, 'utils', 'test_debuginfo.pl')), + 'c-index-test', 'clang-check', 'clang-diff', 'clang-format', 'opt'] + +if config.clang_examples: + tools.append('clang-interpreter') + +# For each occurrence of a clang tool name, replace it with the full path to +# the build directory holding that tool. We explicitly specify the directories +# to search to ensure that we get the tools just built and not some random +# tools that might happen to be in the user's PATH. +tool_dirs = [config.clang_tools_dir, config.llvm_tools_dir] + +llvm_config.add_tool_substitutions(tools, tool_dirs) + +# FIXME: Find nicer way to prohibit this. +config.substitutions.append( + (' clang ', """*** Do not use 'clang' in tests, use '%clang'. ***""")) +config.substitutions.append( + (' clang\+\+ ', """*** Do not use 'clang++' in tests, use '%clangxx'. ***""")) +config.substitutions.append( + (' clang-cc ', + """*** Do not use 'clang-cc' in tests, use '%clang_cc1'. ***""")) +config.substitutions.append( + (' clang -cc1 -analyze ', + """*** Do not use 'clang -cc1 -analyze' in tests, use '%clang_analyze_cc1'. ***""")) +config.substitutions.append( + (' clang -cc1 ', + """*** Do not use 'clang -cc1' in tests, use '%clang_cc1'. ***""")) +config.substitutions.append( + (' %clang-cc1 ', + """*** invalid substitution, use '%clang_cc1'. ***""")) +config.substitutions.append( + (' %clang-cpp ', + """*** invalid substitution, use '%clang_cpp'. ***""")) +config.substitutions.append( + (' %clang-cl ', + """*** invalid substitution, use '%clang_cl'. ***""")) + +config.substitutions.append(('%itanium_abi_triple', + llvm_config.make_itanium_abi_triple(config.target_triple))) +config.substitutions.append(('%ms_abi_triple', + llvm_config.make_msabi_triple(config.target_triple))) +config.substitutions.append(('%resource_dir', builtin_include_dir)) + +# The host triple might not be set, at least if we're compiling clang from +# an already installed llvm. +if config.host_triple and config.host_triple != '@LLVM_HOST_TRIPLE@': + config.substitutions.append(('%target_itanium_abi_host_triple', + '--target=%s' % llvm_config.make_itanium_abi_triple(config.host_triple))) +else: + config.substitutions.append(('%target_itanium_abi_host_triple', '')) + +config.substitutions.append( + ('%src_include_dir', config.clang_src_dir + '/include')) + # Set available features we allow tests to conditionalize on. # if config.clang_default_cxx_stdlib != '': @@ -108,8 +227,6 @@ if platform.system() not in ['Darwin', 'Fuchsia']: config.available_features.add('libgcc') # Case-insensitive file system - - def is_filesystem_case_insensitive(): handle, path = tempfile.mkstemp( prefix='case-test', dir=config.test_exec_root) |