[PATCH] D138895: [clang] Update CWG2635 status

Vlad Serebrennikov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 28 22:18:33 PST 2022


Endill created this revision.
Endill added reviewers: clang-language-wg, erichkeane.
Herald added a project: All.
Endill requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

A follow-up to D138852 <https://reviews.llvm.org/D138852>. Apparently cxx_dr_status.html was changed manually there, since make_cxx_dr_status script doesn't generate the same HTML after that patch landed.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D138895

Files:
  clang/test/CXX/drs/dr26xx.cpp
  clang/www/make_cxx_dr_status


Index: clang/www/make_cxx_dr_status
===================================================================
--- clang/www/make_cxx_dr_status
+++ clang/www/make_cxx_dr_status
@@ -158,18 +158,18 @@
     # This refers to the old ("C++0x") concepts feature, which was not part
     # of any C++ International Standard or Technical Specification.
     continue
-  if dr.issue in (2565, 2628):
+  elif dr.status == 'extension':
     row_style = ' class="open"'
-    avail, avail_style = availability(dr.issue)
-  elif dr.status in ('open', 'concurrency', 'drafting', 'review', 'extension'):
-    # We may have to deal with these some day, but not yet.
+    avail = 'Extension'
+    avail_style = ''
+  elif dr.status in ('open', 'drafting', 'review'):
     row_style = ' class="open"'
-    if dr.status == 'extension':
-      avail = 'Extension'
-    else:
+    avail, avail_style = availability(dr.issue)
+    if avail == 'Unknown':
       avail = 'Not resolved'
-    avail_style = ''
-    assert dr.issue not in status_map, "have status for not-ready dr %s" % dr.issue
+      avail_style = ''
+    if not avail.startswith('Sup') and not avail.startswith('Dup'):
+      count[avail] = count.get(avail, 0) + 1
   else:
     row_style = ''
     avail, avail_style = availability(dr.issue)
Index: clang/test/CXX/drs/dr26xx.cpp
===================================================================
--- clang/test/CXX/drs/dr26xx.cpp
+++ clang/test/CXX/drs/dr26xx.cpp
@@ -29,7 +29,7 @@
 
 }
 
-namespace dr2635 { // dr2635: yes
+namespace dr2635 { // dr2635: 16
 template<typename T>
 concept UnaryC = true;
 template<typename T, typename U>


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D138895.478456.patch
Type: text/x-patch
Size: 1625 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20221129/fa91d175/attachment.bin>


More information about the cfe-commits mailing list