hokein marked 2 inline comments as done. ================ Comment at: unittests/ASTMatchers/ASTMatchersNodeTest.cpp:187 @@ +186,3 @@ + "}", + unresolvedLookupExpr(), true, + "-fno-delayed-template-parsing")); ---------------- alexfh wrote: > thakis wrote: > > Doesn't referencing `bar()` after it's defined help too? If not, this is > > cool too -- maybe add a `/*ExpectMatch=*/` in front of `true` so it's clear > > what the true means. > Yes, please add an argument comment and a FIXME to the code that is known to > be broken with delayed template parsing. Thanks! Done in http://reviews.llvm.org/rL269957.
Repository: rL LLVM http://reviews.llvm.org/D20369 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits