Skip to content

[clang] Add test for CWG2091 "Deducing reference non-type template arguments" #100765

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Aug 5, 2024

Conversation

Endilll
Copy link
Contributor

@Endilll Endilll commented Jul 26, 2024

@llvmbot llvmbot added the clang Clang issues not falling into any other category label Jul 26, 2024
@llvmbot
Copy link
Member

llvmbot commented Jul 26, 2024

@llvm/pr-subscribers-clang

Author: Vlad Serebrennikov (Endilll)

Changes

https://cplusplus.github.io/CWG/issues/2091.html


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

2 Files Affected:

  • (modified) clang/test/CXX/drs/cwg20xx.cpp (+7)
  • (modified) clang/www/cxx_dr_status.html (+1-1)
diff --git a/clang/test/CXX/drs/cwg20xx.cpp b/clang/test/CXX/drs/cwg20xx.cpp
index fdd31845d5e0d..cd2f26360d402 100644
--- a/clang/test/CXX/drs/cwg20xx.cpp
+++ b/clang/test/CXX/drs/cwg20xx.cpp
@@ -401,6 +401,13 @@ namespace cwg2083 { // cwg2083: partial
 #endif
 }
 
+namespace cwg2091 { // cwg2091: 2.7
+template<int &> struct X;
+template<int &N> void f(X<N>&);
+int n;
+void g(X<n> &x) { f(x); }
+} // namespace cwg2091 
+
 namespace cwg2094 { // cwg2094: 5
   struct A { int n; };
   struct B { volatile int n; };
diff --git a/clang/www/cxx_dr_status.html b/clang/www/cxx_dr_status.html
index 937f67981e296..53347234f3c4f 100755
--- a/clang/www/cxx_dr_status.html
+++ b/clang/www/cxx_dr_status.html
@@ -12361,7 +12361,7 @@ <h2 id="cxxdr">C++ defect report implementation status</h2>
     <td><a href="https://cplusplus.github.io/CWG/issues/2091.html">2091</a></td>
     <td>CD4</td>
     <td>Deducing reference non-type template arguments</td>
-    <td class="unknown" align="center">Unknown</td>
+    <td class="full" align="center">Clang 2.7</td>
   </tr>
   <tr id="2092">
     <td><a href="https://cplusplus.github.io/CWG/issues/2092.html">2092</a></td>

@Endilll
Copy link
Contributor Author

Endilll commented Jul 26, 2024

Ideally we should wait for #97200 to land first, then update this PR with the output of make_cxx_dr_status.

@Endilll Endilll added the clang:frontend Language frontend issues, e.g. anything involving "Sema" label Jul 26, 2024
@Endilll
Copy link
Contributor Author

Endilll commented Aug 2, 2024

Ideally we should wait for 97200 to land first, then update this PR with the output of make_cxx_dr_status.

This is done.

Copy link
Contributor

@cor3ntin cor3ntin left a comment

Choose a reason for hiding this comment

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

LGTM

@Endilll Endilll merged commit 1fdc553 into llvm:main Aug 5, 2024
7 checks passed
@Endilll Endilll deleted the cwg2091 branch August 5, 2024 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c++ clang:frontend Language frontend issues, e.g. anything involving "Sema" clang Clang issues not falling into any other category test-suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants