alexfh added inline comments.

================
Comment at: unittests/ASTMatchers/ASTMatchersNodeTest.cpp:187
@@ +186,3 @@
+                                   "}",
+                                   unresolvedLookupExpr(), true,
+                                   "-fno-delayed-template-parsing"));
----------------
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!


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

Reply via email to