[ASTMatchers] Introduce a blockDecl matcher for matching block declarations
ClosedPublic

Authored by george.karpenkov on Wed, May 16, 3:07 PM.

Diff Detail

Repository
rL LLVM
This revision is now accepted and ready to land.Wed, May 16, 3:24 PM
aaron.ballman added inline comments.Wed, May 16, 3:36 PM
lib/ASTMatchers/Dynamic/Registry.cpp
409 ↗(On Diff #147195)

Please keep this in alphabetical order.

NoQ added a subscriber: NoQ.Wed, May 16, 3:38 PM

Yay.

docs/LibASTMatchersReference.html
128 ↗(On Diff #147195)

Blocks aren't Objective-C-specific. Blocks are an Apple extension in normal C as well.

4368 ↗(On Diff #147195)

Shouldn't this be updated to say "or a block"?

This html duplication is weird :o

docs/LibASTMatchersReference.html
128 ↗(On Diff #147195)

OK

4368 ↗(On Diff #147195)

Probably yes. I have no idea how HTML generation works.

lib/ASTMatchers/Dynamic/Registry.cpp
409 ↗(On Diff #147195)

OK

This revision was automatically updated to reflect the committed changes.
NoQ added a comment.Thu, May 17, 1:18 PM

Yep, looks good now!