[PATCH] D114317: [clang-tidy][WIP] Do not run perfect alias checks

Jeroen Van Antwerpen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jan 8 09:38:29 PST 2022


Jeroen added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/ClangTidy.cpp:442
+        if (PrimaryCheckValue != Value) {
+          std::cout << "Alias check \"" << CheckName.str() << "\" of \""
+                    << PrimaryCheckName.str() << "\""
----------------
I'm not sure if you have this in for testing purpose or not, though I really like to have this message. If I would be having 2 aliases active with different options, it would be nice to know about that. So please keep this in!
Preferably, we would only configure it once, though that's dreaming.


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

https://reviews.llvm.org/D114317



More information about the cfe-commits mailing list