erichkeane added a comment.
Thank you so much for doing this. I'm hopeful our hackery can go away sometime
soon.
================
Comment at: clang/test/CXX/drs/dr25xx.cpp:3
-namespace dr2565 { // dr252: 16
+namespace dr2565 { // dr2565: partial
template<typename T>
----------------
If you could add a comment explaining this status for me, I would be grateful:
// We've implemented DR2565 as proposed (that is, a substitution failure in a
requires parameter list causes the requires clause to evaluate as false) as it
seems to make the most sense, and fit best into our implementation. If this
issue gets accepted as-is, this can be marked as complete.
================
Comment at: clang/www/make_cxx_dr_status:161
continue
- if dr.status in ('open', 'concurrency', 'drafting', 'review', 'extension'):
+ if dr.issue in (1432,2565):
+ row_style = ' class="open"'
----------------
I guess this is OK, its a shame we have to custom-tag these, but I guess I'm
1/2 responsible here :/
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D136133/new/
https://reviews.llvm.org/D136133
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
