[PATCH] D97491: [clang-tidy] Deprecate readability-deleted-default check

2021-03-03 Thread Nathan James via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG19aefd2d5dc3: [clang-tidy] Deprecate readability-deleted-default check (authored by njames93). Changed prior to commit: https://reviews.llvm.org/D97491?vs=326449&id=327802#toc Repository: rG LLVM Git

[PATCH] D97491: [clang-tidy] Deprecate readability-deleted-default check

2021-03-01 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. In D97491#2589098 , @Eugene.Zelenko wrote: > In D97491#2589097 , @njames93 wrote: > >> In D97491

[PATCH] D97491: [clang-tidy] Deprecate readability-deleted-default check

2021-02-25 Thread Eugene Zelenko via Phabricator via cfe-commits
Eugene.Zelenko added a comment. In D97491#2589097 , @njames93 wrote: > In D97491#2588835 , @Eugene.Zelenko > wrote: > >> If `Wdefaulted-function-deleted` was introduced in 12.0, I think it's safe >> just to remove

[PATCH] D97491: [clang-tidy] Deprecate readability-deleted-default check

2021-02-25 Thread Nathan James via Phabricator via cfe-commits
njames93 added a comment. In D97491#2588835 , @Eugene.Zelenko wrote: > If `Wdefaulted-function-deleted` was introduced in 12.0, I think it's safe > just to remove `readability-deleted-default`. Wdefaulted-function-deleted was introduced in 8.0 and I thi

[PATCH] D97491: [clang-tidy] Deprecate readability-deleted-default check

2021-02-25 Thread Eugene Zelenko via Phabricator via cfe-commits
Eugene.Zelenko added a comment. If `Wdefaulted-function-deleted` was introduced in 12.0, I think it's safe just to remove `readability-deleted-default`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97491/new/ https://reviews.llvm.org/D97491

[PATCH] D97491: [clang-tidy] Deprecate readability-deleted-default check

2021-02-25 Thread Nathan James via Phabricator via cfe-commits
njames93 created this revision. njames93 added reviewers: aaron.ballman, alexfh. Herald added a subscriber: xazax.hun. njames93 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. ... For removal in next release cycle. The clang warning that do