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

[Jonas Devlieghere via Phabricator via cfe-commits]

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?


Repository:
  rL LLVM

https://reviews.llvm.org/D28768