[PATCH] D53817: [clang-tidy] cppcoreguidelines-macro-usage: print macro names
Jonas Toth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 29 09:03:05 PDT 2018
JonasToth added a comment.
How does this patch change the behaviour for macros without location? They will be diagnosed at the beginning of the TU?
It is probably better to ignore these kind of macros, as there is no way around them (compile time configuration can only be done with macros?!) and therefore warnings for theses macros are false positives. Using `constexpr` constructs does not help either.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D53817
More information about the cfe-commits
mailing list