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

Jonas Devlieghere via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 17 01:11:42 PST 2017

JDevlieghere added a comment.

In https://reviews.llvm.org/D28768#647949, @Prazek wrote:

> Do you have some results from running it on LLVM? If nothing crashes and all fixit are correct then LGTM.

There's still an issue with type narrowing, which is allowed in regular constructor  but not in braced initializer lists. I'd like to skip cases where narrowing occurs, either in the matcher or in the check method, but I haven't figured out how yet.



More information about the cfe-commits mailing list