[PATCH] D138901: [clang] Propely handle tests for open DRs in make_cxx_dr_status

Vlad Serebrennikov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 29 00:21:49 PST 2022


Endill created this revision.
Endill added reviewers: erichkeane, ychen, clang-language-wg.
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 D136133 <https://reviews.llvm.org/D136133>. It was mentioned in #58382 <https://github.com/llvm/llvm-project/issues/58382#issuecomment-1281874338> that there is a need to test for DRs that have not been officially resolved yet. This patch aims to replace original "hackery" with proper handling for such cases. Highlights:

- Availability can be suffixed (further) with "open", "drafting", or "review", e.g. `// dr2565: 16 open`, `// drXXXX: 16 c++17 drafting`
- Checks are implemented to ensure that this suffix corresponds to actual issue status
- Non-resolved DRs are counted (stdout of make_cxx_dr_status)
- No changes made to cxx_dr_status.html
- 'c++20' availability suffix added
- Remove 'concurrency' status since it's no longer on the list of statuses in CWG Active Issues


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D138901

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D138901.478473.patch
Type: text/x-patch
Size: 4180 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20221129/49a4ef0a/attachment.bin>


More information about the cfe-commits mailing list