alexfh added inline comments.
================ Comment at: docs/clang-tidy/checks/bugprone-integer-division.rst:21-24 + // Do not warn, there are signs of deliberateness. + sin(abs(3) / 5); + sin(1 + abs(1 + 7 / 2)); + 1 << 2 / 3; ---------------- I'd argue that the presence of `abs` here doesn't add any signal, since the function has overloads both for integer and floating point types. https://reviews.llvm.org/D35932 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits