This is an archive of the discontinued LLVM Phabricator instance.

[clang] Update DR status to Revision 110
ClosedPublic

Authored by Endill on Nov 28 2022, 10:48 AM.

Details

Reviewers
None
Group Reviewers
Restricted Project
Commits
rGb1a6f2aa89c3: [clang] Update DR status to Revision 110
Summary

Also update a hack in make_cxx_dr_status that handles tests for CWGs that are still open.

Diff Detail

Event Timeline

Endill created this revision.Nov 28 2022, 10:48 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 28 2022, 10:48 AM
Endill requested review of this revision.Nov 28 2022, 10:48 AM
erichkeane added inline comments.
clang/www/make_cxx_dr_status
161

1432 is still listed as 'open', is it not? What did you use to generate the above cxx_dr_status.html? Can you post the link? I'm afraid you've used the wrong file.

Endill added inline comments.Nov 28 2022, 10:59 AM
clang/www/make_cxx_dr_status
161

Revision 110 was published yesterday. 1432 has C++17 status now, and handled perfectly even without this hack.

Endill added a project: Restricted Project.Nov 28 2022, 11:04 AM
Endill set the repository for this revision to rG LLVM Github Monorepo.
Endill added a subscriber: cfe-commits.

This looks good to me, thanks! If you post your "Name <email@whatever.foo>" that you'd like to have this committed as, I can do so (or someone else can beat me to it!).

clang/www/make_cxx_dr_status
161

Ah, I see, thanks! It looks like the version on the Kona wiki was marked as 'open', and there are no motions to accept it, however further look at Kona CWG discussion seems to show that this is believed to have been fixed elsewhere, and is closed because of it.

Here it is: Vlad Serebrennikov <serebrennikov.vladislav@gmail.com>

This revision was not accepted when it landed; it landed in state Needs Review.Nov 28 2022, 11:20 AM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.

Woops, I never marked this as approved apaprently :) I definitely intended to above for anyone keeping track.

Thank you for prompt review