Author: Aiden Grossman Date: 2025-08-01T07:31:20-07:00 New Revision: 6b7a8e9015fad3f73b37e517d9189cbf98feb5d1
URL: https://github.com/llvm/llvm-project/commit/6b7a8e9015fad3f73b37e517d9189cbf98feb5d1 DIFF: https://github.com/llvm/llvm-project/commit/6b7a8e9015fad3f73b37e517d9189cbf98feb5d1.diff LOG: [clang] Remove use of %T from crash-report-modules.m (#151615) This patch removes the use of %T from crash-report-modules.m. %T has been deprecated for about seven years, and is to be avoided as it can lead to races between tests. Posting this one as a separate patch given it was explicitly ported to %T in 5c268f078ddbaa7cc1e9ea0ef658eb90563ff5db. Given 36bb47c70802629df98276ce90b0f8163f959e28 was landed seven years ago to completely disable the test on Windows due to continued failures, I think it should be fine to just leave this test unsupported on Windows. Added: Modified: clang/test/Driver/crash-report-modules.m Removed: ################################################################################ diff --git a/clang/test/Driver/crash-report-modules.m b/clang/test/Driver/crash-report-modules.m index e6d0335337910..7f669dc8ededd 100644 --- a/clang/test/Driver/crash-report-modules.m +++ b/clang/test/Driver/crash-report-modules.m @@ -1,15 +1,12 @@ -// FIXME: Instead of %T/crmdir, it would be nice to just use %t, but the -// filename ran into path length limits for the rm command on some Windows -// bots. -// RUN: rm -rf %T/crmdir -// RUN: mkdir -p %T/crmdir/i %T/crmdir/m +// RUN: rm -rf %t/crmdir +// RUN: mkdir -p %t/crmdir/i %t/crmdir/m -// RUN: env FORCE_CLANG_DIAGNOSTICS_CRASH= TMPDIR=%T/crmdir TEMP=%T/crmdir TMP=%T/crmdir \ +// RUN: env FORCE_CLANG_DIAGNOSTICS_CRASH= TMPDIR=%t/crmdir TEMP=%t/crmdir TMP=%t/crmdir \ // RUN: not %clang -fsyntax-only %s -I %S/Inputs/module -isysroot %/t/i/ \ -// RUN: -fmodules -fmodules-cache-path=%T/crmdir/m/ -DFOO=BAR 2>&1 | FileCheck %s +// RUN: -fmodules -fmodules-cache-path=%t/crmdir/m/ -DFOO=BAR 2>&1 | FileCheck %s -// RUN: FileCheck --check-prefix=CHECKSRC %s -input-file %T/crmdir/crash-report-*.m -// RUN: FileCheck --check-prefix=CHECKSH %s -input-file %T/crmdir/crash-report-*.sh +// RUN: FileCheck --check-prefix=CHECKSRC %s -input-file %t/crmdir/crash-report-*.m +// RUN: FileCheck --check-prefix=CHECKSH %s -input-file %t/crmdir/crash-report-*.sh // REQUIRES: crash-recovery // FIXME: This test creates excessively deep directory hierarchies that cause _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
