[clang-tools-extra] 896575e - [clang] Prefer clang_setup over use_clang
via cfe-commits
cfe-commits at lists.llvm.org
Thu Jul 10 06:14:09 PDT 2025
Author: Aiden Grossman
Date: 2025-07-10T06:14:06-07:00
New Revision: 896575eb74bb59466b50fb51934596d72947ebd5
URL: https://github.com/llvm/llvm-project/commit/896575eb74bb59466b50fb51934596d72947ebd5
DIFF: https://github.com/llvm/llvm-project/commit/896575eb74bb59466b50fb51934596d72947ebd5.diff
LOG: [clang] Prefer clang_setup over use_clang
This patch switches over uses of use_clang to clang_setup to fix a
potential race condition that has been impacting CI.
This is split from the refactoring to ensure I'm not missing anything
major here on the clang-tools-extra side.
This should fix #145703.
Reviewers: AaronBallman, HighCommander4, HerrCai0907, petrhosek, Keenuts
Reviewed By: petrhosek
Pull Request: https://github.com/llvm/llvm-project/pull/147437
Added:
Modified:
clang-tools-extra/clangd/test/lit.cfg.py
clang-tools-extra/test/lit.cfg.py
Removed:
################################################################################
diff --git a/clang-tools-extra/clangd/test/lit.cfg.py b/clang-tools-extra/clangd/test/lit.cfg.py
index 489726bc9429b..8ab4309e337d1 100644
--- a/clang-tools-extra/clangd/test/lit.cfg.py
+++ b/clang-tools-extra/clangd/test/lit.cfg.py
@@ -1,7 +1,7 @@
import lit.llvm
lit.llvm.initialize(lit_config, config)
-lit.llvm.llvm_config.use_clang([], [], required=False)
+lit.llvm.llvm_config.clang_setup()
lit.llvm.llvm_config.use_default_substitutions()
config.name = "Clangd"
diff --git a/clang-tools-extra/test/lit.cfg.py b/clang-tools-extra/test/lit.cfg.py
index 9f64fd3d2ffa2..bc82a14a0aed1 100644
--- a/clang-tools-extra/test/lit.cfg.py
+++ b/clang-tools-extra/test/lit.cfg.py
@@ -42,7 +42,7 @@
config.test_exec_root = os.path.join(config.clang_tools_binary_dir, "test")
# Tools need the same environment setup as clang (we don't need clang itself).
-llvm_config.use_clang(required=False)
+llvm_config.clang_setup()
if config.clang_tidy_staticanalyzer:
config.available_features.add("static-analyzer")
More information about the cfe-commits
mailing list