[cfe-commits] [PATCH] New matcher for MaterializeTemporaryExpression.
Sam Panzer
reviews at llvm-reviews.chandlerc.com
Fri Aug 24 11:09:02 PDT 2012
Comments addressed in next revision.
================
Comment at: include/clang/ASTMatchers/ASTMatchers.h:478
@@ +477,3 @@
+///
+/// Example: Given
+/// struct T {void func()};
----------------
Manuel Klimek wrote:
> Please use \code around the code blocks.
Done.
================
Comment at: unittests/ASTMatchers/ASTMatchersTest.cpp:1253
@@ +1252,3 @@
+TEST(MaterializeTempExpr, MatchesTemporary) {
+ StatementMatcher MaterializeTemp = materializeTempExpr();
+
----------------
Manuel Klimek wrote:
> Is there a reason not to just inline this?
Nope. Inlined.
http://llvm-reviews.chandlerc.com/D20
More information about the cfe-commits
mailing list