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

Dmitry Polukhin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 27 08:38:49 PDT 2020


DmitryPolukhin added a comment.

Yes, it is what I meant as a simpler solution. But please add Lit test even for such trivial option.



================
Comment at: clang-tools-extra/clang-tidy/tool/ClangTidyMain.cpp:337
+
+    Config.assign((*Text)->getBuffer());
+  }
----------------
I suggest creating new local variable with text of the config from `Config` or `ConfigFile` file content i.e. avoid modifying `Config` itself.


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

https://reviews.llvm.org/D89936



More information about the cfe-commits mailing list