[PATCH] D95735: [ASTMatchers] Fix matching after generic top-level matcher

2021-02-02 Thread Stephen Kelly via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGd6a06365cf12: [ASTMatchers] Fix matching after generic top-level matcher (authored by stephenkelly). Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D95735: [ASTMatchers] Fix matching after generic top-level matcher

2021-02-02 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 D95735#2534715 , @steveire wrote: > In D95735#2534400 , @aaron.ballman > wrote: > >> I'm a bi

[PATCH] D95735: [ASTMatchers] Fix matching after generic top-level matcher

2021-02-01 Thread Stephen Kelly via Phabricator via cfe-commits
steveire added a comment. In D95735#2534400 , @aaron.ballman wrote: > I'm a bit confused -- I only get one match when I try out your example: > https://godbolt.org/z/xn9efY Are there other changes in-flight that > necessitate this, or is something else

[PATCH] D95735: [ASTMatchers] Fix matching after generic top-level matcher

2021-02-01 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. I'm a bit confused -- I only get one match when I try out your example: https://godbolt.org/z/xn9efY Are there other changes in-flight that necessitate this, or is something else going on? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https:/

[PATCH] D95735: [ASTMatchers] Fix matching after generic top-level matcher

2021-01-30 Thread Stephen Kelly via Phabricator via cfe-commits
steveire created this revision. steveire added reviewers: aaron.ballman, alexfh. steveire requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. With a matcher like expr(anyOf(integerLiteral(equals(42)), unless(expr( and code such as st