This is an archive of the discontinued LLVM Phabricator instance.

Add a matcher for SubstNonTypeTemplateParmExpr.
ClosedPublic

Authored by alexfh on Jun 27 2014, 11:52 AM.

Diff Detail

Event Timeline

alexfh updated this revision to Diff 10941.Jun 27 2014, 11:52 AM
alexfh retitled this revision from to Add a matcher for SubstNonTypeTemplateParmExpr..
alexfh updated this object.
alexfh edited the test plan for this revision. (Show Details)
alexfh added reviewers: djasper, klimek.
alexfh added a subscriber: Unknown Object (MLST).
djasper accepted this revision.Jun 29 2014, 12:43 PM
djasper edited edge metadata.

Looks good.

include/clang/ASTMatchers/ASTMatchers.h
689

While you are here, can you replace "initList()" with "initListExpr()"?

701

Replace initList() with the correct matcher.

This revision is now accepted and ready to land.Jun 29 2014, 12:43 PM
alexfh added inline comments.Jun 29 2014, 3:24 PM
include/clang/ASTMatchers/ASTMatchers.h
689

Done.

701

Done.

alexfh closed this revision.Jun 29 2014, 3:27 PM

Another review where, from my perspective, the patch itself never made
it to the mailing list - is this just another case of starting a
review, then adding the mailing list to the review?

From inspection of http://reviews.llvm.org/D4327 it looks like
(maybe?) the cfe-commits list was added when the review was created
(at least at the same timestamp), so I'm not sure why the initial
mailing wouldn't've reached the mailing list/my inbox.

klimek edited edge metadata.Jun 30 2014, 8:42 AM

Same here - uiuc blocked all phab mails on June 27th (and only on that day)