[libcxx-commits] [libcxx] [libcxx] [test] Enable gdb_pretty_printer_test.sh.cpp for clang-19 (PR #102893)

[Martin Storsjö via libcxx-commits]

https://github.com/mstorsjo created https://github.com/llvm/llvm-project/pull/102893

With current versions of clang-19 in the CI image, this test does pass.


>From debf7fb1068fe1a406a8c2f08edbbfb114d6f1c6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Martin=20Storsj=C3=B6?= <martin at martin.st>
Date: Mon, 12 Aug 2024 14:10:29 +0300
Subject: [PATCH 1/2] [libcxx] [test] Mark gdb_pretty_printer_test.sh.cpp as
 unsupported on Windows

In practice, this test hasn't been run by any of the Windows
configurations so far, because it has been excluded by the
"UNSUPPORTED: clang-18, clang-19" line - but it does end up
failing if running with e.g. clang-20 on Windows.

As the test fails on more fundamental issues on Windows, mark
it outright unsupported on this platform; this fixes testing
libcxx with a recent nightly version of Clang from git main.
---
 libcxx/test/libcxx/gdb/gdb_pretty_printer_test.sh.cpp | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/libcxx/test/libcxx/gdb/gdb_pretty_printer_test.sh.cpp b/libcxx/test/libcxx/gdb/gdb_pretty_printer_test.sh.cpp
index 4c2b483914f47e..2c8534977febc7 100644
--- a/libcxx/test/libcxx/gdb/gdb_pretty_printer_test.sh.cpp
+++ b/libcxx/test/libcxx/gdb/gdb_pretty_printer_test.sh.cpp
@@ -20,6 +20,9 @@
 // support gdb anymore, favoring lldb instead.
 // UNSUPPORTED: android
 
+// This test doesn't work as such on Windows.
+// UNSUPPORTED: windows
+
 // RUN: %{cxx} %{flags} %s -o %t.exe %{compile_flags} -g %{link_flags}
 // Ensure locale-independence for unicode tests.
 // RUN: env LANG=en_US.UTF-8 %{gdb} -nx -batch -iex "set autoload off" -ex "source %S/../../../utils/gdb/libcxx/printers.py" -ex "python register_libcxx_printer_loader()" -ex "source %S/gdb_pretty_printer_test.py" %t.exe

>From b6dcc31eab9ffa99ac4ceb29faff75b80f409665 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Martin=20Storsj=C3=B6?= <martin at martin.st>
Date: Mon, 12 Aug 2024 15:38:03 +0300
Subject: [PATCH 2/2] [libcxx] [test] Enable gdb_pretty_printer_test.sh.cpp for
 clang-19

With current versions of clang-19 in the CI image, this test does
pass.
---
 libcxx/test/libcxx/gdb/gdb_pretty_printer_test.sh.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libcxx/test/libcxx/gdb/gdb_pretty_printer_test.sh.cpp b/libcxx/test/libcxx/gdb/gdb_pretty_printer_test.sh.cpp
index 2c8534977febc7..0645df0ebd498f 100644
--- a/libcxx/test/libcxx/gdb/gdb_pretty_printer_test.sh.cpp
+++ b/libcxx/test/libcxx/gdb/gdb_pretty_printer_test.sh.cpp
@@ -12,7 +12,7 @@
 // UNSUPPORTED: c++03
 
 // TODO: Investigate these failures which break the CI.
-// UNSUPPORTED: clang-17, clang-18, clang-19
+// UNSUPPORTED: clang-17, clang-18
 
 // The Android libc++ tests are run on a non-Android host, connected to an
 // Android device over adb. gdb needs special support to make this work (e.g.