[clang-tools-extra] Add clang-tidy check to suggest replacement of conditional statement with std::min/std::max (PR #77816)

Piotr Zegar via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 11 11:06:44 PST 2024


================
@@ -0,0 +1,29 @@
+.. title:: clang-tidy - readability-ConditionalToStdMinMax
+
+readability-ConditionalToStdMinMax
+==================================
+
+Replaces certain conditional statements with equivalent std::min or std::max expressions, 
+improving readability and promoting the use of standard library functions.
+
----------------
PiotrZSL wrote:

Add info that this may not be best in performance critical code, as using x = std::min/max may cause additional store that would not happen when if is used, as in such case store would happen only when value need to be changed.

https://github.com/llvm/llvm-project/pull/77816


More information about the cfe-commits mailing list