[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