Page MenuHomePhabricator
Differential D90291 Diff 302213 clang-tools-extra/clangd/test/lit.cfg.py

Changeset View

Standalone View

View Options

clang-tools-extra/clangd/test/lit.cfg.py

import lit.llvm import lit.llvm
lit.llvm.initialize(lit_config, config) lit.llvm.initialize(lit_config, config)
lit.llvm.llvm_config.use_clang() lit.llvm.llvm_config.use_clang()
lit.llvm.llvm_config.use_default_substitutions()
config.name = 'Clangd' config.name = 'Clangd'
config.suffixes = ['.test'] config.suffixes = ['.test']
config.excludes = ['Inputs'] config.excludes = ['Inputs']
config.test_format = lit.formats.ShTest(not lit.llvm.llvm_config.use_lit_shell) config.test_format = lit.formats.ShTest(not lit.llvm.llvm_config.use_lit_shell)
config.test_source_root = config.clangd_source_dir + "/test" config.test_source_root = config.clangd_source_dir + "/test"
config.test_exec_root = config.clangd_binary_dir + "/test" config.test_exec_root = config.clangd_binary_dir + "/test"
# Used to enable tests based on the required targets. Can be queried with e.g. # Used to enable tests based on the required targets. Can be queried with e.g.
# REQUIRES: x86-registered-target # REQUIRES: x86-registered-target
def calculate_arch_features(arch_string): def calculate_arch_features(arch_string):
return [arch.lower() + '-registered-target' for arch in arch_string.split()] return [arch.lower() + '-registered-target' for arch in arch_string.split()]
lit.llvm.llvm_config.feature_config([('--targets-built', lit.llvm.llvm_config.feature_config([('--targets-built',
calculate_arch_features)]) calculate_arch_features)])
# Clangd-specific lit environment. # Clangd-specific lit environment.
config.substitutions.append(('%clangd-benchmark-dir', config.substitutions.append(('%clangd-benchmark-dir',
config.clangd_binary_dir + "/benchmarks")) config.clangd_binary_dir + "/benchmarks"))
kadircetUnsubmitted
Done
ReplyInline Actions

we should rather call lit.llvm.llvm_config.use_default_substitutions(), preferably right after use_clang() on line 4.

sorry for the churn, i thought we were already doing this.

kadircet: we should rather call `lit.llvm.llvm_config.use_default_substitutions()`, preferably right…
if config.clangd_build_xpc: if config.clangd_build_xpc:
config.available_features.add('clangd-xpc-support') config.available_features.add('clangd-xpc-support')
if config.clangd_enable_remote:
config.available_features.add('clangd-remote-index')