[PATCH] D11784: [PATCH] clang-tidy check for incorrect move constructor initializers

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 14 05:44:02 PDT 2015


aaron.ballman added a comment.

In http://reviews.llvm.org/D11784#224437, @alexfh wrote:

> In http://reviews.llvm.org/D11784#224433, @aaron.ballman wrote:
>
> > In http://reviews.llvm.org/D11784#224430, @alexfh wrote:
> >
> > > In http://reviews.llvm.org/D11784#224421, @aaron.ballman wrote:
> > >
> > > > In http://reviews.llvm.org/D11784#224386, @alexfh wrote:
> > > >
> > > > >
> > > >
> > >
> >
> >
> > Ah, interesting -- I was avoiding that because of how fragile it is. Also, it seems to require me grouping the CHECK lines together. e.g.) this fails:
>
>
> FileCheck is rather primitive. It's behavior may confuse at first, but once you get accustomed to it, everything becomes clear.


I've definitely been spoiled by -verify. ;-)

~Aaron


http://reviews.llvm.org/D11784





More information about the cfe-commits mailing list