[PATCH] D55006: [clang] - Simplify tools::SplitDebugName

George Rimar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 28 07:45:38 PST 2018


grimar added a comment.

In D55006#1311400 <https://reviews.llvm.org/D55006#1311400>, @dblaikie wrote:

> In D55006#1311398 <https://reviews.llvm.org/D55006#1311398>, @grimar wrote:
>
> > In D55006#1311391 <https://reviews.llvm.org/D55006#1311391>, @dblaikie wrote:
> >
> > > Cool - thanks! Any chance/way to add a test for this that'd show up sooner than the breakage caused by the previous version?
> >
> >
> > Maybe, I am not sure. I was not able to trigger this without using the `clang-tidy` call yet.
>
>
> Any chance other clang tools might reproduce this? clang-check maybe (sort of the most trivial clang tool)?


I'll try to investigate this tomorrow. Never used it I think :)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55006/new/

https://reviews.llvm.org/D55006





More information about the cfe-commits mailing list