[PATCH] D148260: [clang] Mark CWG2331 as N/A

2023-04-15 Thread Vlad Serebrennikov via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG19ef8e8446df: [clang] Mark CWG2331 as N/A (authored by Endill). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D148260/new/ https://reviews.llvm.org/D148260

[PATCH] D148260: [clang] Mark CWG2331 as N/A

2023-04-14 Thread Shafik Yaghmour via Phabricator via cfe-commits
shafik accepted this revision. shafik added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D148260/new/ https://reviews.llvm.org/D148260 ___

[PATCH] D148260: [clang] Mark CWG2331 as N/A

2023-04-14 Thread Vlad Serebrennikov via Phabricator via cfe-commits
Endill added a comment. In D148260#4266802 , @shafik wrote: > The defect report has two examples even though the first one is commented > incorrectly considering the final resolution. I am sure they are covered in > the test suite in other places but wh

[PATCH] D148260: [clang] Mark CWG2331 as N/A

2023-04-13 Thread Shafik Yaghmour via Phabricator via cfe-commits
shafik added a comment. The defect report has two examples even though the first one is commented incorrectly considering the final resolution. I am sure they are covered in the test suite in other places but why not add them? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D148260: [clang] Mark CWG2331 as N/A

2023-04-13 Thread Vlad Serebrennikov via Phabricator via cfe-commits
Endill created this revision. Endill added reviewers: clang-language-wg, shafik. Herald added a project: All. Endill requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. P1787 : CWG2331 is resolved by defining lookup fro