Skip to content

Conversation

yronglin
Copy link
Contributor

Signed-off-by: yronglin <yronglin777@gmail.com>
@llvmbot
Copy link
Member

llvmbot commented Oct 18, 2025

@llvm/pr-subscribers-clang

Author: None (yronglin)

Changes

Since the following 2 patches was landed, mark P1857R3 as partial implemented.


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

1 Files Affected:

  • (modified) clang/www/cxx_status.html (+8-1)
diff --git a/clang/www/cxx_status.html b/clang/www/cxx_status.html
index a35e50150a2ab..9f4287b68d9ef 100755
--- a/clang/www/cxx_status.html
+++ b/clang/www/cxx_status.html
@@ -926,7 +926,14 @@ <h2 id="cxx20">C++20 implementation status</h2>
       </tr>
       <tr>
         <td><a href="https://wg21.link/p1857r3">P1857R3</a></td>
-        <td class="none" align="center">No</td>
+        <td class="partial" align="center">
+          <details>
+            <summary>>Clang 21(Partial)</summary>
+            The restriction 'A module directive may only appear as
+            the first preprocessing tokens in a file' was supported
+            in clang-21.
+          </details>
+        </td>
       </tr>
       <tr>
         <td><a href="https://wg21.link/p2115r0">P2115R0</a></td>

Copy link
Collaborator

@hubert-reinterpretcast hubert-reinterpretcast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM once comments are addressed.

yronglin and others added 2 commits October 20, 2025 11:30
Co-authored-by: h-vetinari <h.vetinari@gmx.com>
Co-authored-by: Hubert Tong <hubert.reinterpretcast@gmail.com>
Copy link
Member

@ChuanqiXu9 ChuanqiXu9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Out of curiosity, which parts are not implemented?

@yronglin
Copy link
Contributor Author

yronglin commented Oct 20, 2025

Out of curiosity, which parts are not implemented?

All the rest parts will be supported in #107168. 🤣

@yronglin yronglin merged commit e65522e into llvm:main Oct 20, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clang Clang issues not falling into any other category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants