On Wed, Feb 3, 2016 at 3:57 PM, Kim Gräsman <kim.gras...@gmail.com> wrote: > On Mon, Feb 1, 2016 at 4:32 PM, Aaron Ballman <aaron.ball...@gmail.com> wrote: >> >> ================ >> Comment at: clang-tidy/readability/RedundantControlFlowCheck.cpp:60 >> @@ +59,3 @@ >> + if (const auto *Return = dyn_cast<ReturnStmt>(*last)) >> + issueDiagnostic(Result, Block, Return->getSourceRange(), >> + RedundantReturnDiag); >> ---------------- >> It is the LLVM coding style (though I don't personally care for it), and you >> are right -- a clang-tidy check in the LLVM module wouldn't be amiss. :-) > > Isn't this: > http://clang.llvm.org/extra/clang-tidy/checks/readability-braces-around-statements.html?
Yes, that seems to be it. Probably could have an alias in the LLVM umbrella that points here. ~Aaron _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
