cor3ntin added a comment. Looks good except I think it makes more sense to put the new test just before the first test
================ Comment at: clang/test/CXX/drs/dr13xx.cpp:25 } namespace dr1310 { // dr1310: 5 ---------------- I think it would make more sense to put the new test here Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122674/new/ https://reviews.llvm.org/D122674 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits