aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

LGTM aside from some commenting requests.



================
Comment at: include/clang/AST/ASTTypeTraits.h:44
+enum TraversalKind {
+  /// Will traverse any child nodes.
+  TK_AsIs,
----------------
s/any/all ?


================
Comment at: include/clang/AST/ASTTypeTraits.h:47
+
+  /// Will not traverse implicit casts and parentheses.
+  TK_IgnoreImplicitCastsAndParentheses
----------------
Can you update the comment to more clearly state what is skipped, because it's 
more than just implicit casts and parentheses?

If we can find a better name for the traversal kind, then great, but I can't 
think of one. `IgnoreRandomImplicitJunk` doesn't seem like an improvement, 
despite it being more accurate. :-P


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61836/new/

https://reviews.llvm.org/D61836



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

Reply via email to