[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