[PATCH] D93861: [clang-tidy][NFC] Split up some headers

2020-12-28 Thread Nathan James via Phabricator via cfe-commits
njames93 updated this revision to Diff 313878. njames93 marked 3 inline comments as done. njames93 added a comment. Fix clang-tidy lint and comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93861/new/ https://reviews.llvm.org/D93861 Files:

[PATCH] D93861: [clang-tidy][NFC] Split up some headers

2020-12-28 Thread Eugene Zelenko via Phabricator via cfe-commits
Eugene.Zelenko added inline comments. Comment at: clang-tools-extra/clang-tidy/ClangTidyContext.h:9 -#ifndef LLVM_CLANG_TOOLS_EXTRA_CLANG_TIDY_CLANGTIDYDIAGNOSTICCONSUMER_H -#define LLVM_CLANG_TOOLS_EXTRA_CLANG_TIDY_CLANGTIDYDIAGNOSTICCONSUMER_H Should header g

[PATCH] D93861: [clang-tidy][NFC] Split up some headers

2020-12-28 Thread Nathan James via Phabricator via cfe-commits
njames93 created this revision. njames93 added a reviewer: aaron.ballman. Herald added subscribers: usaxena95, kadircet, arphaman, kbarton, xazax.hun, nemanjai. njames93 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Extract the ClangTidy