[PATCH] D16183: Added CheckName field to YAML report
Alexander Kornienko via cfe-commits
cfe-commits at lists.llvm.org
Tue Jan 19 18:30:07 PST 2016
alexfh added a comment.
In http://reviews.llvm.org/D16183#329475, @Elijah_Th wrote:
> What kind of wrapper should it be?
> I was thinking of this kind:
>
> class ExtendedReplacement : public Replacement {
No, I meant a different thing: serialize ClangTidyError or make a specialized structure for this purpose. Apart from replacements it contains the diagnostic message, diagnostic location, and the check name. We'll just need to teach clang-apply-replacements to read this structure.
http://reviews.llvm.org/D16183
More information about the cfe-commits
mailing list