Page MenuHomePhabricator

Add utility for testing if we're matching nodes AsIs

Authored by steveire on Tue, Nov 10, 1:47 AM.

Diff Detail

Event Timeline

steveire created this revision.Tue, Nov 10, 1:47 AM
Herald added a project: Restricted Project. · View Herald TranscriptTue, Nov 10, 1:47 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
steveire requested review of this revision.Tue, Nov 10, 1:47 AM
aaron.ballman accepted this revision.Tue, Nov 10, 5:20 AM

LGTM, thanks!


I don't insist, but I do wonder if we want to inline the definition in the header file rather than put it in the implementation file so that call sites have an easier time inlining the functionality.

This revision is now accepted and ready to land.Tue, Nov 10, 5:20 AM
steveire added inline comments.Tue, Nov 10, 11:25 AM

If I inline it I get

error: invalid use of incomplete type ‘class clang::ASTContext’

I think there was a drive at some point to use ASTContext less in header files.

aaron.ballman added inline comments.Tue, Nov 10, 11:27 AM

Thanks for checking!

This revision was landed with ongoing or failed builds.Tue, Nov 10, 11:28 AM
This revision was automatically updated to reflect the committed changes.