Consider using std::min and std::max instead of fmin and fmax

[dimitry-ishenko]

fmin and fmax result in a function call on GCC and MSVC (or extra instructions on Clang).

Whereas, std::min and std::max map to single XMM instruction. I haven't bench-marked it, but it should definitely give performance boost.

Proof: https://godbolt.org/z/1jbPqbvGx

[dimitry-ishenko]

NB: On Windows, you will need to define NOMINMAX somewhere at the top (or in the CMake file).

[eloj]

I'll just note that this is one of those things that are affected by -ffast-math.

Specifically -ffinite-math-only seems to eliminate the call for GCC and clang at least.