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

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 29 06:22:21 PST 2022


aaron.ballman accepted this revision as: aaron.ballman.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

LGTM though my Python skills are lacking, so additional eyeballs on that would be appreciated.



================
Comment at: clang/www/make_cxx_dr_status:187-189
+      assert unresolved_status == dr.status, \
+             "Issue %s is marked '%s', which differs from CWG index status '%s'" \
+             % (dr.issue, unresolved_status, dr.status)
----------------
Thank you for this! It resolves the biggest concern I had with our markings getting out of sync with the issues list.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D138901/new/

https://reviews.llvm.org/D138901



More information about the cfe-commits mailing list