[libcxx-commits] [libcxx] 03f6f48 - [libc++] Move a few tests into more correct places (#147557)

[via libcxx-commits]

Author: Nikolas Klauser
Date: 2025-07-10T20:55:07+02:00
New Revision: 03f6f48b73725279f35ec05a5235dda793328e1c

URL: https://github.com/llvm/llvm-project/commit/03f6f48b73725279f35ec05a5235dda793328e1c
DIFF: https://github.com/llvm/llvm-project/commit/03f6f48b73725279f35ec05a5235dda793328e1c.diff

LOG: [libc++] Move a few tests into more correct places (#147557)

Added: 
    libcxx/test/extensions/all/cstddef.compile.pass.cpp
    libcxx/test/extensions/clang/clang_modules_include.gen.py
    libcxx/test/extensions/clang/lit.local.cfg
    libcxx/test/extensions/libcxx/include_as_c.sh.cpp
    libcxx/test/extensions/libcxx/libcpp_version.gen.py
    libcxx/test/extensions/libcxx/lit.local.cfg
    libcxx/test/extensions/libcxx/no_assert_include.gen.py
    libcxx/test/std/double_include.gen.py
    libcxx/test/std/header_inclusions.gen.py

Modified: 
    

Removed: 
    libcxx/test/libcxx/clang_modules_include.gen.py
    libcxx/test/libcxx/double_include.gen.py
    libcxx/test/libcxx/header_inclusions.gen.py
    libcxx/test/libcxx/include_as_c.sh.cpp
    libcxx/test/libcxx/language.support/support.types/cstddef.compile.pass.cpp
    libcxx/test/libcxx/libcpp_version.gen.py
    libcxx/test/libcxx/no_assert_include.gen.py
    libcxx/test/selftest/lit.local.cfg


################################################################################
diff  --git a/libcxx/test/libcxx/language.support/support.types/cstddef.compile.pass.cpp b/libcxx/test/extensions/all/cstddef.compile.pass.cpp
similarity index 99%
rename from libcxx/test/libcxx/language.support/support.types/cstddef.compile.pass.cpp
rename to libcxx/test/extensions/all/cstddef.compile.pass.cpp
index 514353a103029..c6f670d7bfc06 100644
--- a/libcxx/test/libcxx/language.support/support.types/cstddef.compile.pass.cpp
+++ b/libcxx/test/extensions/all/cstddef.compile.pass.cpp
@@ -11,6 +11,7 @@
 // appear to provide that behavior too.
 
 #include <cstddef>
+
 #include "test_macros.h"
 
 using Ptr
diff T = ::ptr
diff _t;

diff  --git a/libcxx/test/libcxx/clang_modules_include.gen.py b/libcxx/test/extensions/clang/clang_modules_include.gen.py
similarity index 100%
rename from libcxx/test/libcxx/clang_modules_include.gen.py
rename to libcxx/test/extensions/clang/clang_modules_include.gen.py

diff  --git a/libcxx/test/extensions/clang/lit.local.cfg b/libcxx/test/extensions/clang/lit.local.cfg
new file mode 100644
index 0000000000000..b0a1c7d9b17e4
--- /dev/null
+++ b/libcxx/test/extensions/clang/lit.local.cfg
@@ -0,0 +1,4 @@
+
+# Only libc++ supports clang-specific extensions
+if "stdlib=libc++" not in config.available_features:
+    config.unsupported = True

diff  --git a/libcxx/test/libcxx/include_as_c.sh.cpp b/libcxx/test/extensions/libcxx/include_as_c.sh.cpp
similarity index 100%
rename from libcxx/test/libcxx/include_as_c.sh.cpp
rename to libcxx/test/extensions/libcxx/include_as_c.sh.cpp

diff  --git a/libcxx/test/libcxx/libcpp_version.gen.py b/libcxx/test/extensions/libcxx/libcpp_version.gen.py
similarity index 93%
rename from libcxx/test/libcxx/libcpp_version.gen.py
rename to libcxx/test/extensions/libcxx/libcpp_version.gen.py
index b30623fe2c388..ebeab777c934f 100644
--- a/libcxx/test/libcxx/libcpp_version.gen.py
+++ b/libcxx/test/extensions/libcxx/libcpp_version.gen.py
@@ -6,7 +6,7 @@
 #
 # ===----------------------------------------------------------------------===##
 
-# Test that all headers define the _LIBCPP_VERSION macro.
+# Test that all public headers define the _LIBCPP_VERSION macro.
 
 # RUN: %{python} %s %{libcxx-dir}/utils
 

diff  --git a/libcxx/test/extensions/libcxx/lit.local.cfg b/libcxx/test/extensions/libcxx/lit.local.cfg
new file mode 100644
index 0000000000000..2b128105558b2
--- /dev/null
+++ b/libcxx/test/extensions/libcxx/lit.local.cfg
@@ -0,0 +1,3 @@
+
+if "stdlib=libc++" not in config.available_features:
+    config.unsupported = True

diff  --git a/libcxx/test/libcxx/no_assert_include.gen.py b/libcxx/test/extensions/libcxx/no_assert_include.gen.py
similarity index 100%
rename from libcxx/test/libcxx/no_assert_include.gen.py
rename to libcxx/test/extensions/libcxx/no_assert_include.gen.py

diff  --git a/libcxx/test/selftest/lit.local.cfg b/libcxx/test/selftest/lit.local.cfg
deleted file mode 100644
index 4467d8070cc70..0000000000000
--- a/libcxx/test/selftest/lit.local.cfg
+++ /dev/null
@@ -1,5 +0,0 @@
-# The tests in this directory need to run Python
-import shlex
-import sys
-
-config.substitutions.append(("%{python}", shlex.quote(sys.executable)))

diff  --git a/libcxx/test/libcxx/double_include.gen.py b/libcxx/test/std/double_include.gen.py
similarity index 93%
rename from libcxx/test/libcxx/double_include.gen.py
rename to libcxx/test/std/double_include.gen.py
index f58e72f94a353..fcf3b9a8fa2e0 100644
--- a/libcxx/test/libcxx/double_include.gen.py
+++ b/libcxx/test/std/double_include.gen.py
@@ -28,6 +28,9 @@
 {lit_header_restrictions.get(header, '')}
 {lit_header_undeprecations.get(header, '')}
 
+// We're using compiler-specific flags in this test
+// REQUIRES: (gcc || clang)
+
 // RUN: %{{cxx}} -c %s -o %t.first.o %{{flags}} %{{compile_flags}}
 // RUN: %{{cxx}} -c %s -o %t.second.o -DWITH_MAIN %{{flags}} %{{compile_flags}}
 // RUN: %{{cxx}} -o %t.exe %t.first.o %t.second.o %{{flags}} %{{link_flags}}

diff  --git a/libcxx/test/libcxx/header_inclusions.gen.py b/libcxx/test/std/header_inclusions.gen.py
similarity index 91%
rename from libcxx/test/libcxx/header_inclusions.gen.py
rename to libcxx/test/std/header_inclusions.gen.py
index e00cf180d17ad..8ff93810069fa 100644
--- a/libcxx/test/libcxx/header_inclusions.gen.py
+++ b/libcxx/test/std/header_inclusions.gen.py
@@ -48,6 +48,9 @@
 //--- {header}.compile.pass.cpp
 // UNSUPPORTED: FROZEN-CXX03-HEADERS-FIXME
 
+// TODO: This is currently a libc++-specific way of testing the includes, but is a requirement for all implementation
+// REQUIRES: stdlib=libc++
+
 {lit_header_restrictions.get(header, '')}
 {lit_header_undeprecations.get(header, '')}