[PATCH] D80499: Remove obsolete ignore*() matcher uses

Stephen Kelly via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 17 06:50:20 PST 2020


steveire added a comment.

In D80499#2362028 <https://reviews.llvm.org/D80499#2362028>, @steveire wrote:

>> ! In D80499#2353187 <https://reviews.llvm.org/D80499#2353187>, @alexfh wrote:
>>  You should be ready for back and forth with this change, if users hit widespread issues not caught by tests. Maybe splitting it into separate pieces and involving check authors where practical may be reasonable.
>
> Ok, I'll do that when https://reviews.llvm.org/D80961 and https://reviews.llvm.org/D82278 are merged.

@alexfh This is done in D91302 <https://reviews.llvm.org/D91302> and D91303 <https://reviews.llvm.org/D91303>. Could you review?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80499



More information about the cfe-commits mailing list