[PATCH] D20369: [ASTMatcher] Fix a ASTMatcher test failure on Windows.

Haojian Wu via cfe-commits cfe-commits at lists.llvm.org
Wed May 18 09:57:00 PDT 2016


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





More information about the cfe-commits mailing list