This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy][NFC] Split up some headers
Needs ReviewPublic

Authored by njames93 on Dec 28 2020, 8:18 AM.

Details

Reviewers
aaron.ballman
Summary

Extract the ClangTidyOptionsProvider and ClangTidyContext into their own headers.
The main purpose of this change is to reduce Dependancies in ClangTidyCheck.

Diff Detail

Event Timeline

njames93 created this revision.Dec 28 2020, 8:18 AM
njames93 requested review of this revision.Dec 28 2020, 8:18 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 28 2020, 8:18 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
Eugene.Zelenko added inline comments.
clang-tools-extra/clang-tidy/ClangTidyContext.h
9

Should header guard just renamed?

196

Please add newline.

278

Should header guard just renamed?

njames93 updated this revision to Diff 313878.Dec 28 2020, 11:43 AM
njames93 marked 3 inline comments as done.

Fix clang-tidy lint and comments.