Author: bd1976bris Date: 2025-09-17T20:16:38+01:00 New Revision: 6ad0f8e16b223707827efbf2cd0825b793b1db14
URL: https://github.com/llvm/llvm-project/commit/6ad0f8e16b223707827efbf2cd0825b793b1db14 DIFF: https://github.com/llvm/llvm-project/commit/6ad0f8e16b223707827efbf2cd0825b793b1db14.diff LOG: [DTLTO][TEST] Make Clang driver tests even more robust (#159418) Remove XFAILs for llvm-driver. 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. Added: Modified: clang/test/Driver/DTLTO/dtlto.c clang/test/Driver/DTLTO/ps5-dtlto.c Removed: ################################################################################ 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. _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
