jansvoboda11 removed a reviewer: jansvoboda11.
jansvoboda11 added inline comments.
Herald added a subscriber: jansvoboda11.
Comment at: clang/lib/Frontend/CompilerInvocation.cpp:640
+StringRef CheckList = A->getValue();
+Opts.TidyChecks.emplace_back(CheckList);
+ }
alexfh added a comment.
Artem, could you set the repository to `rG LLVM Github Monorepo` when uploading
patches as mentioned in
https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface
? This way you'll allow pre-merge checks to run.
In D95403#2534714
aaron.ballman added a comment.
Thank you for working on this, I think it's fantastic effort!
> I'll make sure to implement (and hopefully test) a reasonable behavior for
> all various combinations of CMake flags (eg., clang-tidy enabled/disabled,
> static analyzer enabled/disabled,
NoQ added inline comments.
Comment at: clang/lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp:54-60
+#include "../../clang-tools-extra/clang-tidy/ClangTidyCheck.h"
+#include "../../clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.h"
+#include
alexfh added inline comments.
Comment at: clang/lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp:54-60
+#include "../../clang-tools-extra/clang-tidy/ClangTidyCheck.h"
+#include "../../clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.h"
+#include
NoQ updated this revision to Diff 319514.
NoQ added a comment.
Unforget to `git add` tests.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95403/new/
https://reviews.llvm.org/D95403
Files:
clang/include/clang/Driver/Options.td
NoQ created this revision.
NoQ added reviewers: vsavchenko, xazax.hun, martong, Szelethus,
baloghadamsoftware, Charusso, steakhal, gribozavr2, alexfh, aaron.ballman.
Herald added subscribers: dang, ASDenysPetrov, phosek, dkrupp, donat.nagy,
mikhail.ramalho, a.sidorin, rnkovacs, szepet, mgorny.