[clang] [DTLTO][TEST] Fix Clang driver test failing on some build bots (PR #148908)
via cfe-commits
cfe-commits at lists.llvm.org
Tue Jul 15 10:46:19 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang
Author: bd1976bris (bd1976bris)
<details>
<summary>Changes</summary>
Not all builds name the compiler executable `clang`. For example, the Fuchsia build bots use `llvm` as their single toolchain executable name, as they combine everything together in a busybot-style binary.
Update the Clang driver test to simply check that a non-empty path is provided for the `--thinlto-remote-compiler` argument, rather than hardcoding the executable name. The cross-project test will verify that the path is valid later.
Should fix #<!-- -->147265.
---
Full diff: https://github.com/llvm/llvm-project/pull/148908.diff
1 Files Affected:
- (modified) clang/test/Driver/DTLTO/dtlto.c (+1-1)
``````````diff
diff --git a/clang/test/Driver/DTLTO/dtlto.c b/clang/test/Driver/DTLTO/dtlto.c
index d72e487e706b2..96795d9a4e6a4 100644
--- a/clang/test/Driver/DTLTO/dtlto.c
+++ b/clang/test/Driver/DTLTO/dtlto.c
@@ -10,7 +10,7 @@
// FORWARD: ld.lld
// FORWARD-SAME: "--thinlto-distributor=d.exe"
-// FORWARD-SAME: "--thinlto-remote-compiler={{.*}}clang{{[^\"]*}}"
+// FORWARD-SAME: "--thinlto-remote-compiler={{[^"]+}}"
// FORWARD-SAME: "--thinlto-distributor-arg=a1"
// FORWARD-SAME: "--thinlto-distributor-arg=a2"
// FORWARD-SAME: "--thinlto-distributor-arg=a3"
``````````
</details>
https://github.com/llvm/llvm-project/pull/148908
More information about the cfe-commits
mailing list