Eugene.Zelenko added a comment. Please mention this check in docs/ReleaseNotes.rst (in alphabetical order).
================ Comment at: clang-tools-extra/docs/clang-tidy/checks/performance-type-promotion-in-math-fn.rst:6 + +Finds calls to math.h functions with implicit float to double promotions. + ---------------- Please enclose math.h, float and double into ``. Will be good idea to mention cmath too. ================ Comment at: clang-tools-extra/docs/clang-tidy/checks/performance-type-promotion-in-math-fn.rst:8 + +For example, warns on `::sin(0.f)`, because this funciton's parameter is a +double. You probably meant to call `::sinf(0.f)`, or maybe `std::sin(0.f)`. ---------------- Please enclose functions into ``. Same in other places. ================ Comment at: clang-tools-extra/docs/clang-tidy/checks/performance-type-promotion-in-math-fn.rst:9 +For example, warns on `::sin(0.f)`, because this funciton's parameter is a +double. You probably meant to call `::sinf(0.f)`, or maybe `std::sin(0.f)`. ---------------- Please fix double space. https://reviews.llvm.org/D27284 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
