[PATCH] D27284: [ClangTidy] Add new performance-type-promotion-in-math-fn check.
Justin Lebar via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Dec 1 18:09:00 PST 2016
jlebar updated this revision to Diff 80011.
jlebar added a comment.
Suggest std::foo instead of ::foof.
I spoke with rtrieu IRL and he suggested I do this to match what he did in
SemaChecking.cpp for calls to "abs". Seems reasonable to me. This also lets
me sidestep the question of lexing the qualifiers, which is nice.
https://reviews.llvm.org/D27284
Files:
clang-tools-extra/clang-tidy/performance/CMakeLists.txt
clang-tools-extra/clang-tidy/performance/PerformanceTidyModule.cpp
clang-tools-extra/clang-tidy/performance/TypePromotionInMathFnCheck.cpp
clang-tools-extra/clang-tidy/performance/TypePromotionInMathFnCheck.h
clang-tools-extra/docs/clang-tidy/checks/list.rst
clang-tools-extra/docs/clang-tidy/checks/performance-type-promotion-in-math-fn.rst
clang-tools-extra/test/clang-tidy/performance-type-promotion-in-math-fn.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D27284.80011.patch
Type: text/x-patch
Size: 24146 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161202/b8961fd1/attachment-0001.bin>
More information about the cfe-commits
mailing list