[clang] [NFC] [C++] [Modules] Mark P2788 as implemented and add test (PR #147138)
via cfe-commits
cfe-commits at lists.llvm.org
Sat Jul 5 03:25:21 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang
Author: Ashwin Kishin Banwari (kish1n)
<details>
<summary>Changes</summary>
This is already implemented.
---
Full diff: https://github.com/llvm/llvm-project/pull/147138.diff
2 Files Affected:
- (added) clang/test/SemaCXX/P2788.cpp (+17)
- (modified) clang/www/cxx_status.html (+1-1)
``````````diff
diff --git a/clang/test/SemaCXX/P2788.cpp b/clang/test/SemaCXX/P2788.cpp
new file mode 100644
index 0000000000000..a9184e1521e8e
--- /dev/null
+++ b/clang/test/SemaCXX/P2788.cpp
@@ -0,0 +1,17 @@
+// RUN: rm -rf %t
+// RUN: split-file %s %t
+
+
+// RUN: %clang_cc1 -std=c++20 -verify -emit-module-interface %t/B.cpp -o %t/B.pcm
+// RUN: %clang_cc1 -std=c++20 -verify -emit-module-interface %t/A.cpp -fmodule-file=A:B=%t/B.pcm -o %t/A.pcm
+
+//--- A.cpp
+// expected-no-diagnostics
+export module A;
+import :B;
+export int x = dimensions + 1;
+
+//--- B.cpp
+// expected-no-diagnostics
+export module A:B;
+const int dimensions=3;
diff --git a/clang/www/cxx_status.html b/clang/www/cxx_status.html
index 831f79f7cf17a..c13d273c9f863 100755
--- a/clang/www/cxx_status.html
+++ b/clang/www/cxx_status.html
@@ -947,7 +947,7 @@ <h2 id="cxx20">C++20 implementation status</h2>
</tr>
<tr> <!-- from Issaquah 2023 -->
<td><a href="https://wg21.link/P2788R0">P2788R0</a> (<a href="#dr">DR</a>)</td>
- <td class="none" align="center">No</td>
+ <td class="full" align="center">Clang 17</td>
</tr>
<tr>
<td>Coroutines</td>
``````````
</details>
https://github.com/llvm/llvm-project/pull/147138
More information about the cfe-commits
mailing list