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

Stephen Kelly via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Oct 25 06:05:40 PDT 2020


steveire added a subscriber: alexfh.
steveire added a comment.

@alexfh This change is based on the behavior of AST Matchers being changed to ignore invisible/implicit AST nodes by default. As the default was not changed in the end, this patch would need to be updated to add `traverse(TK_IgnoreUnlessSpelledInSource)` wrapping around the matchers.

Before I update the patch to add that, do you have any feedback?


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