This is an archive of the discontinued LLVM Phabricator instance.

Change default traversal in AST Matchers to ignore invisible nodes
ClosedPublic

Authored by steveire on Jan 10 2020, 11:52 AM.

Event Timeline

steveire created this revision.Jan 10 2020, 11:52 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 10 2020, 11:52 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
aaron.ballman accepted this revision.May 15 2020, 6:28 AM

LGTM aside from some documentation nits.

clang/docs/ReleaseNotes.rst
255 ↗(On Diff #263552)

is -> are

Also, I think we typically use single spacing after the full stop in the release notes.

257–259 ↗(On Diff #263552)

We also have release notes for clang-tools-extra that should be updated for the changes to clang-query.

This revision is now accepted and ready to land.May 15 2020, 6:28 AM
This revision was automatically updated to reflect the committed changes.