[PATCH] D28768: [clang-tidy] Add check 'modernize-return-braced-init-list'

Jonas Devlieghere via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 16 04:36:12 PST 2017

JDevlieghere marked 2 inline comments as done.
JDevlieghere added a comment.

In https://reviews.llvm.org/D28768#647203, @malcolm.parsons wrote:

> In https://reviews.llvm.org/D28768#647198, @JDevlieghere wrote:
> > In https://reviews.llvm.org/D28768#647177, @malcolm.parsons wrote:
> >
> > > What happens if the function has `auto` as the return type?
> >
> >
> > Nothing, the constructor is left untouched.
> Please add a test.

I wanted to do that but it seems that the test script hard codes it to C++11, and the `auto` return type is a C++14 feature. Maybe I'm mistaken though, but I didn't manage to get it working, even when appending `-- -std=c++14` to the RUN command. Am I missing something?



More information about the cfe-commits mailing list