aaron.ballman added inline comments.
================
Comment at: include/clang/AST/ASTTypeTraits.h:81
+ /// \{
+ /// Return the AST node kind of this ASTNodeKind.
----------------
These markings are a bit strange, can you explain them to me?
================
Comment at: include/clang/ASTMatchers/ASTMatchers.h:6389
+/// Given OpenMP directive, matches the first clause (out of all specified),
+/// that matches InnerMatcher.
----------------
Given OpenMP -> Given an OpenMP
================
Comment at: lib/ASTMatchers/Dynamic/Registry.cpp:509-512
+ REGISTER_MATCHER(hasClause);
+ REGISTER_MATCHER(ompDefaultClause);
+ REGISTER_MATCHER(isNoneKind);
+ REGISTER_MATCHER(isAllowedToContainClause);
----------------
Please keep these alphabetically sorted.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57112/new/
https://reviews.llvm.org/D57112
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
