Szelethus accepted this revision. Szelethus added a comment. This revision is now accepted and ready to land.
LGTM! You did check whether a missing doc field will actually trigger this error, right? ================ Comment at: clang/utils/TableGen/ClangSACheckersEmitter.cpp:82 + PrintFatalError(R.getLoc(), + "missing Documentation for " + getCheckerFullName(&R)); + ---------------- We might as well give an extra hand, and state that a `Documentation<> field` is missing. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122244/new/ https://reviews.llvm.org/D122244 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
