aaron.ballman accepted this revision. aaron.ballman added a comment. There's one small wording nit with the diagnostic, but once that's resolved, LGTM as well.
================ Comment at: clang-tidy/modernize/ReturnBracedInitListCheck.cpp:65 + + auto Diag = diag(Loc, "to avoid repeating the return type from the " + "declaration; use a braced initializer list instead"); ---------------- This diagnostic reads strangely. I think you should drop the "to" at the start of the sentence. Repository: rL LLVM https://reviews.llvm.org/D28768 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits