Endill added a comment.

In D156057#4527787 <https://reviews.llvm.org/D156057#4527787>, @rZhBoYao wrote:

> BTW, I am not sure if CWG2571 <http://wg21.link/cwg2571> is implemented by 
> @cor3ntin? If so, can we mark it as done on 
> https://clang.llvm.org/cxx_dr_status.html#2571? This patch handles the 
> warning around it tho.

As a reminder, we don't change `cxx_dr_status.html` directly. Instead, we add a 
special comment in `clang/test/CXX/drs/dr25xx.cpp` (for the case of CWG2571), 
which should be accompanied by test case(s) if possible. You can follow the 
pattern of existing tests. After that `clang/www/make_cxx_dr_status` script 
should be run to update the HTML.


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

https://reviews.llvm.org/D156057

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to