[llvm-branch-commits] [libcxx] [libc++][C++03] Remove XFAILs from the non-frozen libc++-specific tests (PR #144101)

2025-08-26 Thread Nikolas Klauser via llvm-branch-commits

https://github.com/philnik777 updated 
https://github.com/llvm/llvm-project/pull/144101

>From 219ba169840842e131bdb11cf18ee7730a6d1bab Mon Sep 17 00:00:00 2001
From: Nikolas Klauser 
Date: Fri, 13 Jun 2025 17:49:01 +0200
Subject: [PATCH] [libc++][C++03] Remove XFAILs from the non-frozen
 libc++-specific tests

---
 libcxx/test/libcxx/algorithms/half_positive.pass.cpp   | 2 --
 libcxx/test/libcxx/algorithms/vectorization.compile.pass.cpp   | 2 --
 .../assertions/customize_verbose_abort.link-time.pass.cpp  | 2 --
 libcxx/test/libcxx/assertions/default_verbose_abort.pass.cpp   | 2 --
 libcxx/test/libcxx/assertions/modes/none.pass.cpp  | 2 --
 libcxx/test/libcxx/assertions/single_expression.pass.cpp   | 2 --
 .../atomics.types.operations.req/atomic_fetch_add.verify.cpp   | 2 --
 .../atomic_fetch_add_explicit.verify.cpp   | 2 --
 .../atomics.types.operations.req/atomic_fetch_sub.verify.cpp   | 2 --
 .../atomic_fetch_sub_explicit.verify.cpp   | 2 --
 libcxx/test/libcxx/clang_tidy.gen.py   | 3 ---
 .../containers/associative/tree_balance_after_insert.pass.cpp  | 2 --
 .../libcxx/containers/associative/tree_left_rotate.pass.cpp| 2 --
 libcxx/test/libcxx/containers/associative/tree_remove.pass.cpp | 2 --
 .../libcxx/containers/associative/tree_right_rotate.pass.cpp   | 2 --
 .../test/libcxx/containers/container_traits.compile.pass.cpp   | 2 --
 libcxx/test/libcxx/containers/unord/next_pow2.pass.cpp | 2 --
 libcxx/test/libcxx/containers/unord/next_prime.pass.cpp| 2 --
 .../libcxx/experimental/fexperimental-library.compile.pass.cpp | 2 --
 .../string.streams/stringbuf/const_sso_buffer.pass.cpp | 2 --
 libcxx/test/libcxx/iterators/aliasing_iterator.pass.cpp| 2 --
 libcxx/test/libcxx/iterators/bounded_iter/arithmetic.pass.cpp  | 2 --
 libcxx/test/libcxx/iterators/bounded_iter/comparison.pass.cpp  | 2 --
 .../test/libcxx/iterators/bounded_iter/pointer_traits.pass.cpp | 2 --
 .../test/libcxx/iterators/bounded_iter/types.compile.pass.cpp  | 2 --
 .../iterators/contiguous_iterators.conv.compile.pass.cpp   | 2 --
 libcxx/test/libcxx/iterators/contiguous_iterators.verify.cpp   | 2 --
 .../iterator.primitives/iterator.operations/prev.verify.cpp| 2 --
 .../language.support/support.dynamic/libcpp_deallocate.sh.cpp  | 2 --
 libcxx/test/libcxx/memory/allocation_guard.pass.cpp| 2 --
 libcxx/test/libcxx/memory/swap_allocator.pass.cpp  | 2 --
 libcxx/test/libcxx/numerics/bit.ops.pass.cpp   | 2 --
 libcxx/test/libcxx/numerics/clamp_to_integral.pass.cpp | 2 --
 .../libcxx/numerics/complex.number/cmplx.over.pow.pass.cpp | 2 --
 .../strings/basic.string/string.capacity/max_size.pass.cpp | 2 --
 .../test/libcxx/strings/c.strings/constexpr_memmove.pass.cpp   | 2 --
 libcxx/test/libcxx/system_reserved_names.gen.py| 2 --
 libcxx/test/libcxx/transitive_includes.gen.py  | 2 --
 libcxx/test/libcxx/type_traits/datasizeof.compile.pass.cpp | 2 --
 libcxx/test/libcxx/type_traits/desugars_to.compile.pass.cpp| 2 --
 libcxx/test/libcxx/type_traits/is_constant_evaluated.pass.cpp  | 2 --
 libcxx/test/libcxx/type_traits/is_replaceable.compile.pass.cpp | 2 --
 .../type_traits/is_trivially_comparable.compile.pass.cpp   | 2 --
 .../type_traits/is_trivially_relocatable.compile.pass.cpp  | 2 --
 libcxx/test/libcxx/utilities/exception_guard.odr.sh.cpp| 2 --
 .../function.objects/refwrap/desugars_to.compile.pass.cpp  | 2 --
 libcxx/test/libcxx/utilities/is_pointer_in_range.pass.cpp  | 2 --
 libcxx/test/libcxx/utilities/is_valid_range.pass.cpp   | 2 --
 .../libcxx/utilities/meta/is_referenceable.compile.pass.cpp| 2 --
 libcxx/test/libcxx/utilities/meta/meta_base.pass.cpp   | 2 --
 libcxx/test/libcxx/utilities/no_destroy.pass.cpp   | 2 --
 .../utilities/utility/private_constructor_tag.compile.pass.cpp | 2 --
 libcxx/test/selftest/test_macros.pass.cpp  | 2 --
 53 files changed, 107 deletions(-)

diff --git a/libcxx/test/libcxx/algorithms/half_positive.pass.cpp 
b/libcxx/test/libcxx/algorithms/half_positive.pass.cpp
index 88a18e8592921..40f4697419572 100644
--- a/libcxx/test/libcxx/algorithms/half_positive.pass.cpp
+++ b/libcxx/test/libcxx/algorithms/half_positive.pass.cpp
@@ -11,8 +11,6 @@
 // __half_positive divides an integer number by 2 as unsigned number for known 
types.
 // It can be an important optimization for lower bound, for example.
 
-// XFAIL: FROZEN-CXX03-HEADERS-FIXME
-
 #include <__algorithm/half_positive.h>
 #include 
 #include 
diff --git a/libcxx/test/libcxx/algorithms/vectorization.compile.pass.cpp 
b/libcxx/test/libcxx/algorithms/vectorization.compile.pass.cpp
index 872c49a35dd76..109fc78507cc4 100644
--- a/libcxx/test/libcxx/algorithms/vectorization.compile.pass.cpp
+++ b/libcxx/test/libcxx/algorithms/vectorization.compile.pass.cpp
@@ -

[llvm-branch-commits] [libcxx] [libc++][C++03] Remove XFAILs from the non-frozen libc++-specific tests (PR #144101)

2025-06-18 Thread Louis Dionne via llvm-branch-commits

https://github.com/ldionne approved this pull request.


https://github.com/llvm/llvm-project/pull/144101
___
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits


[llvm-branch-commits] [libcxx] [libc++][C++03] Remove XFAILs from the non-frozen libc++-specific tests (PR #144101)

2025-06-13 Thread via llvm-branch-commits

github-actions[bot] wrote:




:warning: Python code formatter, darker found issues in your code. :warning:



You can test this locally with the following command:


``bash
darker --check --diff -r HEAD~1...HEAD 
libcxx/test/libcxx/clang_modules_include.gen.py 
libcxx/test/libcxx/clang_tidy.gen.py 
libcxx/test/libcxx/header_inclusions.gen.py 
libcxx/test/libcxx/system_reserved_names.gen.py 
libcxx/test/libcxx/transitive_includes.gen.py
``





View the diff from darker here.


``diff
--- clang_tidy.gen.py   2025-06-13 15:49:01.00 +
+++ clang_tidy.gen.py   2025-06-13 15:53:41.838897 +
@@ -17,11 +17,12 @@
 import sys
 sys.path.append(sys.argv[1])
 from libcxx.header_information import lit_header_restrictions, 
lit_header_undeprecations, public_headers
 
 for header in public_headers:
-  print(f"""\
+print(
+f"""\
 //--- {header}.sh.cpp
 
 // REQUIRES: has-clang-tidy
 
 // The GCC compiler flags are not always compatible with clang-tidy.
@@ -32,6 +33,7 @@
 
 // TODO: run clang-tidy with modules enabled once they are supported
 // RUN: %{{clang-tidy}} %s --warnings-as-errors=* -header-filter=.* 
--config-file=%{{libcxx-dir}}/.clang-tidy 
--load=%{{test-tools-dir}}/clang_tidy_checks/libcxx-tidy.plugin -- 
-Wweak-vtables %{{compile_flags}} -fno-modules
 
 #include <{header}>
-""")
+"""
+)

``




https://github.com/llvm/llvm-project/pull/144101
___
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits


[llvm-branch-commits] [libcxx] [libc++][C++03] Remove XFAILs from the non-frozen libc++-specific tests (PR #144101)

2025-06-13 Thread Nikolas Klauser via llvm-branch-commits

https://github.com/philnik777 created 
https://github.com/llvm/llvm-project/pull/144101

The tests in `libcxx/test/libcxx` aren't run against the frozen headers 
anymore, so we can remove any XFAILs in them.

This is part of https://discourse.llvm.org/t/rfc-freezing-c-03-headers-in-libc.


>From e080572b8168260ecb4c8b2be39111d579056f74 Mon Sep 17 00:00:00 2001
From: Nikolas Klauser 
Date: Fri, 13 Jun 2025 17:49:01 +0200
Subject: [PATCH] [libc++][C++03] Remove XFAILs from the non-frozen
 libc++-specific tests

---
 libcxx/test/libcxx/algorithms/half_positive.pass.cpp   | 2 --
 libcxx/test/libcxx/algorithms/vectorization.compile.pass.cpp   | 2 --
 .../assertions/customize_verbose_abort.link-time.pass.cpp  | 2 --
 libcxx/test/libcxx/assertions/default_verbose_abort.pass.cpp   | 2 --
 libcxx/test/libcxx/assertions/modes/none.pass.cpp  | 2 --
 libcxx/test/libcxx/assertions/single_expression.pass.cpp   | 2 --
 .../atomics.types.operations.req/atomic_fetch_add.verify.cpp   | 2 --
 .../atomic_fetch_add_explicit.verify.cpp   | 2 --
 .../atomics.types.operations.req/atomic_fetch_sub.verify.cpp   | 2 --
 .../atomic_fetch_sub_explicit.verify.cpp   | 2 --
 libcxx/test/libcxx/clang_modules_include.gen.py| 2 --
 libcxx/test/libcxx/clang_tidy.gen.py   | 3 ---
 .../containers/associative/tree_balance_after_insert.pass.cpp  | 2 --
 .../containers/associative/tree_key_value_traits.pass.cpp  | 2 --
 .../libcxx/containers/associative/tree_left_rotate.pass.cpp| 2 --
 libcxx/test/libcxx/containers/associative/tree_remove.pass.cpp | 2 --
 .../libcxx/containers/associative/tree_right_rotate.pass.cpp   | 2 --
 .../containers/associative/unord.map/abi.compile.pass.cpp  | 2 --
 .../containers/associative/unord.set/abi.compile.pass.cpp  | 2 --
 .../test/libcxx/containers/container_traits.compile.pass.cpp   | 2 --
 libcxx/test/libcxx/containers/unord/key_value_traits.pass.cpp  | 2 --
 libcxx/test/libcxx/containers/unord/next_pow2.pass.cpp | 2 --
 libcxx/test/libcxx/containers/unord/next_prime.pass.cpp| 2 --
 libcxx/test/libcxx/depr/depr.c.headers/extern_c.pass.cpp   | 2 --
 .../libcxx/experimental/fexperimental-library.compile.pass.cpp | 2 --
 libcxx/test/libcxx/header_inclusions.gen.py| 1 -
 .../string.streams/stringbuf/const_sso_buffer.pass.cpp | 2 --
 libcxx/test/libcxx/iterators/aliasing_iterator.pass.cpp| 2 --
 libcxx/test/libcxx/iterators/bounded_iter/arithmetic.pass.cpp  | 2 --
 libcxx/test/libcxx/iterators/bounded_iter/comparison.pass.cpp  | 2 --
 .../test/libcxx/iterators/bounded_iter/pointer_traits.pass.cpp | 2 --
 .../test/libcxx/iterators/bounded_iter/types.compile.pass.cpp  | 2 --
 .../iterators/contiguous_iterators.conv.compile.pass.cpp   | 2 --
 libcxx/test/libcxx/iterators/contiguous_iterators.verify.cpp   | 2 --
 .../iterator.primitives/iterator.operations/prev.verify.cpp| 2 --
 .../language.support/support.dynamic/libcpp_deallocate.sh.cpp  | 2 --
 libcxx/test/libcxx/memory/allocation_guard.pass.cpp| 2 --
 libcxx/test/libcxx/memory/swap_allocator.pass.cpp  | 2 --
 libcxx/test/libcxx/numerics/bit.ops.pass.cpp   | 2 --
 libcxx/test/libcxx/numerics/clamp_to_integral.pass.cpp | 2 --
 .../libcxx/numerics/complex.number/cmplx.over.pow.pass.cpp | 2 --
 libcxx/test/libcxx/selftest/test_macros.pass.cpp   | 2 --
 .../strings/basic.string/string.capacity/max_size.pass.cpp | 2 --
 .../test/libcxx/strings/c.strings/constexpr_memmove.pass.cpp   | 2 --
 libcxx/test/libcxx/system_reserved_names.gen.py| 2 --
 libcxx/test/libcxx/transitive_includes.gen.py  | 2 --
 libcxx/test/libcxx/type_traits/datasizeof.compile.pass.cpp | 2 --
 libcxx/test/libcxx/type_traits/desugars_to.compile.pass.cpp| 2 --
 libcxx/test/libcxx/type_traits/is_constant_evaluated.pass.cpp  | 2 --
 libcxx/test/libcxx/type_traits/is_replaceable.compile.pass.cpp | 2 --
 .../type_traits/is_trivially_comparable.compile.pass.cpp   | 2 --
 .../type_traits/is_trivially_relocatable.compile.pass.cpp  | 2 --
 libcxx/test/libcxx/utilities/exception_guard.odr.sh.cpp| 2 --
 .../function.objects/refwrap/desugars_to.compile.pass.cpp  | 2 --
 libcxx/test/libcxx/utilities/is_pointer_in_range.pass.cpp  | 2 --
 libcxx/test/libcxx/utilities/is_valid_range.pass.cpp   | 2 --
 .../libcxx/utilities/meta/is_referenceable.compile.pass.cpp| 2 --
 libcxx/test/libcxx/utilities/meta/meta_base.pass.cpp   | 2 --
 libcxx/test/libcxx/utilities/no_destroy.pass.cpp   | 2 --
 libcxx/test/libcxx/utilities/template.bitset/includes.pass.cpp | 2 --
 .../utilities/utility/private_constructor_tag.compile.pass.cpp | 2 --
 61 files changed, 122 deletions(-)

diff --git a/libcxx/test/libcxx/algorithms/half_positive.pass.cpp 
b/libcxx/test/libcxx/algorithms/half_positiv