aaron.ballman added a comment.

Missing tests and changes to Registry.cpp for dynamic matchers.

Also, do you want to add `isInstantiationDependent()` at the same time, given 
the relationship with the other two matchers?



================
Comment at: include/clang/ASTMatchers/ASTMatchers.h:777
 
+/// Matches expressions that are type-dependant because the template type
+/// is not yet instantiated.
----------------
s/dependant/dependent


================
Comment at: include/clang/ASTMatchers/ASTMatchers.h:787
+///   }
+/// \endcode
+AST_MATCHER(Expr, isTypeDependent) { return Node.isTypeDependent(); }
----------------
Your example should include the matcher syntax.


================
Comment at: include/clang/ASTMatchers/ASTMatchers.h:790
+
+/// Matches expression that are value-dependant because they contain a
+/// non-type template parameter.
----------------
s/dependant/dependent


Repository:
  rC Clang

https://reviews.llvm.org/D51880



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to