[cfe-dev] [clang-tidy] Addition info in YAML report

Manuel Klimek via cfe-dev cfe-dev at lists.llvm.org
Mon Dec 21 23:51:57 PST 2015


Adding the check id sounds like a useful feature.

On Mon, Dec 21, 2015, 1:44 PM Ilia Gromov via cfe-dev <
cfe-dev at lists.llvm.org> wrote:

> Hi,
>
> clang-tidy saves a YAML report when the option '-export-fixes=...> is used.
>
> ---
> MainSourceFile:  ''
> Replacements:
>    - FilePath:        /home/ilia/clang/sandbox/main.cpp
>      Offset:          388
>      Length:          8
>      ReplacementText: '// TODO(ilia): '
> ...
>
> This information is sufficient to apply generated replacements later.
> However, there is no information about a check which had found this
> warning.
>
> Is there a way to know check ID for this replacement?
>
>
> PS:
> In clang-modernize this problem was solved with a workaround:
> When in "serialize-replacements" mode, clang-modernize can't inspect
> sources more than for 1 check ID.
> So, when I run
>
>      ./clang-modernize -serialize-replacements
> -serialize-dir=/tmp/modernize/add-override112233 /tmp/source.cpp
>
> I'm sure that a YAML file in /tmp/modernize/add-override112233 is for
> "add-override" check.Repeat this for all 6 checks and, as a result, you
> can group replacements by check ID.
> clang-tidy allows to specify any number of check IDs when saving to
> YAML. And it has way more checks than 6. So, this workaround won't work
> well in case of clang-tidy
>
> --
>
> Thanks,
> Ilia Gromov
> _______________________________________________
> cfe-dev mailing list
> cfe-dev at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20151222/9e12888f/attachment-0001.html>


More information about the cfe-dev mailing list