[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