Skip to content

Conversation

philnik777
Copy link
Contributor

This test only fails because it includes <__config>. Switch to using
<__cxx03/__config> instead to fix the issue.

This test only fails because it includes `<__config>`. Switch to using
`<__cxx03/__config>` instead to fix the issue.
@philnik777 philnik777 marked this pull request as ready for review August 26, 2025 14:26
@philnik777 philnik777 requested a review from a team as a code owner August 26, 2025 14:26
@llvmbot llvmbot added the libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi. label Aug 26, 2025
@philnik777 philnik777 merged commit 4b30563 into llvm:main Aug 26, 2025
69 of 73 checks passed
@philnik777 philnik777 deleted the cxx03_fix_system_reserved_names branch August 26, 2025 14:26
@llvmbot
Copy link
Member

llvmbot commented Aug 26, 2025

@llvm/pr-subscribers-libcxx

Author: Nikolas Klauser (philnik777)

Changes

This test only fails because it includes &lt;__config&gt;. Switch to using
&lt;__cxx03/__config&gt; instead to fix the issue.


Full diff: https://github.com/llvm/llvm-project/pull/155385.diff

1 Files Affected:

  • (modified) libcxx/test/libcxx-03/system_reserved_names.gen.py (+1-3)
diff --git a/libcxx/test/libcxx-03/system_reserved_names.gen.py b/libcxx/test/libcxx-03/system_reserved_names.gen.py
index a4f2928eda332..43c008823a85e 100644
--- a/libcxx/test/libcxx-03/system_reserved_names.gen.py
+++ b/libcxx/test/libcxx-03/system_reserved_names.gen.py
@@ -29,10 +29,8 @@
 {lit_header_restrictions.get(header, '')}
 {lit_header_undeprecations.get(header, '')}
 
-// UNSUPPORTED: FROZEN-CXX03-HEADERS-FIXME
-
 // This is required to detect the platform we're building for below.
-#include <__config>
+#include <__cxx03/__config>
 
 #define SYSTEM_RESERVED_NAME This name should not be used in libc++
 

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants