aaron.ballman added a comment. I did find a few more spurious semicolons with the testing code as well.
If you need me to commit this on your behalf, I'm happy to do so, though it won't happen until tomorrow or Wed. Or, if you have commit privs, feel free to commit with the fixed test case. ================ Comment at: unittests/ASTMatchers/ASTMatchersNarrowingTest.cpp:1995-1997 + EXPECT_TRUE(matches("void x(int val = 0) {};", + parmVarDecl(hasDefaultArgument()))); + EXPECT_TRUE(notMatches("void x(int val) {};", ---------------- Spurious semicolons in here as well (sorry, I missed that before). https://reviews.llvm.org/D40261 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits