aaron.ballman added inline comments.
================ Comment at: include/clang/ASTMatchers/ASTMatchers.h:6421 +/// +/// Prerequisite: the executable directive must not be standalone directive. +/// ---------------- What happens if this prereq is not met? Does the matcher return false, or does it do something worse (like assert)? ================ Comment at: lib/ASTMatchers/Dynamic/Registry.cpp:513-515 + REGISTER_MATCHER(isStandaloneDirective); + REGISTER_MATCHER(isOMPStructuredBlock); + REGISTER_MATCHER(hasStructuredBlock); ---------------- Alphabetical order, please. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59463/new/ https://reviews.llvm.org/D59463 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits