NoQ added a comment. Nice!
There's usually some bureaucracy when creating new matchers, i.e. there should be documentation and unittests covering them. ================ Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:5099 +/// matches a FunctionType if the type includes the GNU no return attribute +AST_MATCHER(FunctionType, typeHasNoReturnAttr) { + return Node.getNoReturnAttr(); ---------------- Can we make a polymorphic matcher `hasNoReturnAttr` that can accept either decl or type? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128314/new/ https://reviews.llvm.org/D128314 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits