[PATCH] D89936: [clang-tidy] adding "--clang-tidy-config=" to specify custom config file

[Hiral via Phabricator via cfe-commits]

Hiralo added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/tool/ClangTidyMain.cpp:330
   if (!Config.empty()) {
     if (llvm::ErrorOr<ClangTidyOptions> ParsedConfig =
             parseConfiguration(Config)) {
----------------
DmitryPolukhin wrote:
> I think you can make this option much simpler if you just read file content 
> and use it or `Config` here. No changes in 
> clang-tools-extra/clang-tidy/ClangTidyOptions.h/.cpp will be required.
Sorry, didn't get. Can you please elaborate/pseudocode?



CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D89936/new/

https://reviews.llvm.org/D89936

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits