[clang] [DTLTO][TEST] Make Clang driver tests even more robust (PR #159418)
via cfe-commits
cfe-commits at lists.llvm.org
Wed Sep 17 11:30:38 PDT 2025
https://github.com/bd1976bris created https://github.com/llvm/llvm-project/pull/159418
Remove XFAILs for Multicall. DTLTO is still incompatible with llvm-driver, but these tests now pass after #159151.
Modify a missed regex to use filename.py (missed in #159151).
Tighten overly greedy regexes to prevent spurious failures.
>From c066b8db9b8ac477febd38f4131ac2941479e710 Mon Sep 17 00:00:00 2001
From: Dunbobbin <Ben.Dunbobbin at sony.com>
Date: Wed, 17 Sep 2025 19:23:12 +0100
Subject: [PATCH] [DTLTO][TEST] Make Clang driver tests even more robust
Remove XFAILs for Multicall. DTLTO is still incompatible with
llvm-driver, but these tests now pass after #159151.
Modify a missed regex to use filename.py (missed in #159151).
Tighten overly greedy regexes to prevent spurious failures.
---
clang/test/Driver/DTLTO/dtlto.c | 8 +++-----
clang/test/Driver/DTLTO/ps5-dtlto.c | 7 ++-----
2 files changed, 5 insertions(+), 10 deletions(-)
diff --git a/clang/test/Driver/DTLTO/dtlto.c b/clang/test/Driver/DTLTO/dtlto.c
index 66299c890e4b3..f31b635feed96 100644
--- a/clang/test/Driver/DTLTO/dtlto.c
+++ b/clang/test/Driver/DTLTO/dtlto.c
@@ -1,8 +1,5 @@
// REQUIRES: lld
-/// https://github.com/llvm/llvm-project/issues/159125.
-// XFAIL: llvm-driver
-
/// Check DTLTO options are forwarded to the linker.
/// Check that options are forwarded as expected with --thinlto-distributor=.
@@ -15,7 +12,7 @@
// FORWARD: filename.py:[[CLANG:.*]]
// FORWARD: ld.lld
// FORWARD-SAME: "--thinlto-distributor=d.exe"
-// FORWARD-SAME: "--thinlto-remote-compiler={{.*}}[[CLANG]]"
+// FORWARD-SAME: "--thinlto-remote-compiler={{[^"]*}}[[CLANG]]"
// FORWARD-SAME: "--thinlto-distributor-arg=a1"
// FORWARD-SAME: "--thinlto-distributor-arg=a2"
// FORWARD-SAME: "--thinlto-distributor-arg=a3"
@@ -39,9 +36,10 @@
// RUN: FileCheck %s --input-file=%t_default.log --check-prefix=DEFAULT \
// RUN: --implicit-check-not=distributor --implicit-check-not=remote-compiler
+// DEFAULT: filename.py:[[CLANG:.*]]
// DEFAULT: ld.lld
// DEFAULT-SAME: "--thinlto-distributor=d.exe"
-// DEFAULT-SAME: "--thinlto-remote-compiler={{[^"]+}}"
+// DEFAULT-SAME: "--thinlto-remote-compiler={{[^"]*}}[[CLANG]]"
/// Check that nothing is forwarded when the compiler is not in LTO mode, and that
/// appropriate unused option warnings are issued.
diff --git a/clang/test/Driver/DTLTO/ps5-dtlto.c b/clang/test/Driver/DTLTO/ps5-dtlto.c
index b52765db5b1c7..4c10a026f944b 100644
--- a/clang/test/Driver/DTLTO/ps5-dtlto.c
+++ b/clang/test/Driver/DTLTO/ps5-dtlto.c
@@ -1,8 +1,5 @@
// REQUIRES: lld
-/// https://github.com/llvm/llvm-project/issues/159125.
-// XFAIL: llvm-driver
-
/// Check DTLTO options are forwarded to the linker.
/// Check that options are forwarded as expected with --thinlto-distributor=.
@@ -15,7 +12,7 @@
// FORWARD: filename.py:[[CLANG:.*]]
// FORWARD: prospero-lld
// FORWARD-SAME: "--thinlto-distributor=d.exe"
-// FORWARD-SAME: "--thinlto-remote-compiler={{.*}}[[CLANG]]"
+// FORWARD-SAME: "--thinlto-remote-compiler={{[^"]*}}[[CLANG]]"
// FORWARD-SAME: "--thinlto-distributor-arg=a1"
// FORWARD-SAME: "--thinlto-distributor-arg=a2"
// FORWARD-SAME: "--thinlto-distributor-arg=a3"
@@ -42,7 +39,7 @@
// DEFAULT: filename.py:[[CLANG:.*]]
// DEFAULT: prospero-lld
// DEFAULT-SAME: "--thinlto-distributor=d.exe"
-// DEFAULT-SAME: "--thinlto-remote-compiler={{.*}}[[CLANG]]"
+// DEFAULT-SAME: "--thinlto-remote-compiler={{[^"]*}}[[CLANG]]"
/// Check that the arguments are forwarded unconditionally even when the
/// compiler is not in LTO mode.
More information about the cfe-commits
mailing list