Using minMaxOp for reductions is fragile

When reducing min/max values, the minMaxOp attempts to be extra smart and also allow reduction of two MinMax values as well:

template<class T>
struct minMaxOp
{
...
     MinMax<T> operator()(const MinMax<T>& x, const MinMax<T>& y) const
     {
         return MinMax<T>(x).add(y);  // Same as (x + y)
     }
};

However, this can sometime confuse the compiler. Better to use plusOp<MinMax<T>>() for binary reductions.

assigned to @mark

mentioned in commit d5e3f063

mentioned in commit a81543c5

mentioned in commit 14b2302b

mentioned in commit 25ca8e22

fixed with !728 (merged)

closed

mentioned in issue #3336 (closed)