[PATCH] D59318: [clang-tidy] add an overload for diag method that allows users to provide a diagnostic name rather than using the check name when building a diagnostic.
Harshal Lehri via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Mar 17 10:22:52 PDT 2019
htl abandoned this revision.
htl added a comment.
In D59318#1430829 <https://reviews.llvm.org/D59318#1430829>, @JonasToth wrote:
> What is the reason you want this change to happen? I think this gives the chance to create inconsistencies which we should avoid.
Sorry this was a bit rushed. I was trying to pass some metadata in the diagnostic name so that another tool that uses the diagnostic name can perform some conditional evaluation based on the data. I will try coming up with a better plan and design for such a change.
Thank you
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59318/new/
https://reviews.llvm.org/D59318
More information about the cfe-commits
mailing list