[PATCH] D21303: [clang-tidy] Adds performance-returning-type check.

2017-04-23 Thread Jakub Staroń via Phabricator via cfe-commits
staronj abandoned this revision. staronj marked 9 inline comments as done. staronj added a comment. Because of https://reviews.llvm.org/D21619 the check is no longer required. https://reviews.llvm.org/D21303 ___ cfe-commits mailing list cfe-commits@

[PATCH] D27806: [clang-tidy] Add obvious-invalid-range

2016-12-16 Thread Jakub Staroń via Phabricator via cfe-commits
staronj added inline comments. Comment at: clang-tidy/misc/InvalidRangeCheck.cpp:19 + +const std::string CXX_AlgorithmNames = +"std::for_each; std::find; std::find_if; std::find_end; " Remove std::fill_n (http://en.cppreference.com/w/cpp/algorithm/fill_n) -

[PATCH] D27700: [clang-tidy] refactor ExprSequence out of misc-use-after-move check

2016-12-13 Thread Jakub Staroń via Phabricator via cfe-commits
staronj added inline comments. Comment at: clang-tidy/utils/ExprSequence.cpp:52 + +bool isDescendantOrEqual(const Stmt *Descendant, const Stmt *Ancestor, + ASTContext *Context) { Shouldn't isDescendantOrEqual be static or in inline namespa