Re: [PATCH] D17060: [Clang] Fix Clang-tidy readability-redundant-control-flow warnings; other minor fixes

2016-02-10 Thread Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL260414: Fix some Clang-tidy readability-redundant-control-flow warnings; other minor… (authored by eugenezelenko). Changed prior to commit: http://reviews.llvm.org/D17060?vs=47415=47492#toc

Re: [PATCH] D17060: [Clang] Fix Clang-tidy readability-redundant-control-flow warnings; other minor fixes

2016-02-10 Thread Hans Wennborg via cfe-commits
hans accepted this revision. hans added a comment. This revision is now accepted and ready to land. Looks good to me. Repository: rL LLVM http://reviews.llvm.org/D17060 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D17060: [Clang] Fix Clang-tidy readability-redundant-control-flow warnings; other minor fixes

2016-02-09 Thread Eugene Zelenko via cfe-commits
Eugene.Zelenko created this revision. Eugene.Zelenko added reviewers: hans, aaron.ballman. Eugene.Zelenko added a subscriber: cfe-commits. Eugene.Zelenko set the repository for this revision to rL LLVM. I checked this patch on my own build on RHEL 6. Regressions were OK. Repository: rL LLVM

Re: [PATCH] D17060: [Clang] Fix Clang-tidy readability-redundant-control-flow warnings; other minor fixes

2016-02-09 Thread Eugene Zelenko via cfe-commits
Eugene.Zelenko updated this revision to Diff 47415. Eugene.Zelenko added a comment. More fixes. Repository: rL LLVM http://reviews.llvm.org/D17060 Files: include/clang/Analysis/Analyses/ThreadSafetyUtil.h lib/ARCMigrate/ObjCMT.cpp lib/AST/ASTDiagnostic.cpp lib/AST/CommentLexer.cpp