[clang] [NFC] [C++] [Modules] Mark P2615 as implemented and add test (PR #147135)

2025-07-07 Thread Chuanqi Xu via cfe-commits 

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

[clang] [NFC] [C++] [Modules] Mark P2615 as implemented and add test (PR #147135)

2025-07-06 Thread Chuanqi Xu via cfe-commits 

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

LGTM.  Thanks. A small suggestion is, it is better to paste the link of the 
paper. So that I don't have to find it my self.

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

[clang] [NFC] [C++] [Modules] Mark P2615 as implemented and add test (PR #147135)

2025-07-05 Thread Ashwin Kishin Banwari via cfe-commits 

kish1n wrote:

@ChuanqiXu9 

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

[clang] [NFC] [C++] [Modules] Mark P2615 as implemented and add test (PR #147135)

2025-07-05 Thread via cfe-commits 

llvmbot wrote:




@llvm/pr-subscribers-clang

Author: Ashwin Kishin Banwari (kish1n)


Changes

This is already implemented. See godbolt: https://godbolt.org/z/1ra4Ka8od

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


2 Files Affected:

- (added) clang/test/SemaCXX/P2615.cpp (+11) 
- (modified) clang/www/cxx_status.html (+1-1) 


``diff
diff --git a/clang/test/SemaCXX/P2615.cpp b/clang/test/SemaCXX/P2615.cpp
new file mode 100644
index 0..fed177c21bc79
--- /dev/null
+++ b/clang/test/SemaCXX/P2615.cpp
@@ -0,0 +1,11 @@
+// RUN: rm -rf %t
+// RUN: split-file %s %t
+
+
+// RUN: %clang_cc1 -std=c++20 -verify -fsyntax-only %t/A.cpp
+
+//--- A.cpp
+// expected-no-diagnostics
+export module A;
+export namespace N {int x = 42;}
+export using namespace N;
diff --git a/clang/www/cxx_status.html b/clang/www/cxx_status.html
index 831f79f7cf17a..16e90a4dbb209 100755
--- a/clang/www/cxx_status.html
+++ b/clang/www/cxx_status.html
@@ -943,7 +943,7 @@ C++20 implementation status
   

 https://wg21.link/P2615R1";>P2615R1 (DR)
-No
+Clang 17
   

 https://wg21.link/P2788R0";>P2788R0 (DR)

``




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

[clang] [NFC] [C++] [Modules] Mark P2615 as implemented and add test (PR #147135)

2025-07-05 Thread Ashwin Kishin Banwari via cfe-commits 

https://github.com/kish1n created 
https://github.com/llvm/llvm-project/pull/147135

This is already implemented. See godbolt: https://godbolt.org/z/1ra4Ka8od

>From 088b7af5c463b46af8261033ea778fa383a138c7 Mon Sep 17 00:00:00 2001
From: Ashwin Banwari 
Date: Sat, 5 Jul 2025 01:55:52 -0700
Subject: [PATCH] mark P2615 as implemented

---
 clang/test/SemaCXX/P2615.cpp | 11 +++
 clang/www/cxx_status.html|  2 +-
 2 files changed, 12 insertions(+), 1 deletion(-)
 create mode 100644 clang/test/SemaCXX/P2615.cpp

diff --git a/clang/test/SemaCXX/P2615.cpp b/clang/test/SemaCXX/P2615.cpp
new file mode 100644
index 0..fed177c21bc79
--- /dev/null
+++ b/clang/test/SemaCXX/P2615.cpp
@@ -0,0 +1,11 @@
+// RUN: rm -rf %t
+// RUN: split-file %s %t
+
+
+// RUN: %clang_cc1 -std=c++20 -verify -fsyntax-only %t/A.cpp
+
+//--- A.cpp
+// expected-no-diagnostics
+export module A;
+export namespace N {int x = 42;}
+export using namespace N;
diff --git a/clang/www/cxx_status.html b/clang/www/cxx_status.html
index 831f79f7cf17a..16e90a4dbb209 100755
--- a/clang/www/cxx_status.html
+++ b/clang/www/cxx_status.html
@@ -943,7 +943,7 @@ C++20 implementation status
   

 https://wg21.link/P2615R1";>P2615R1 (DR)
-No
+Clang 17
   

 https://wg21.link/P2788R0";>P2788R0 (DR)

___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits