[Bug libstdc++/100237] Unnecessary std::move in ranges::min, ranges::max and ranges::minmax

[ppalka at gcc dot gnu.org via Gcc-bugs]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100237

Patrick Palka <ppalka at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Last reconfirmed|                            |2021-04-23
             Status|UNCONFIRMED                 |NEW
     Ever confirmed|0                           |1
                 CC|                            |ppalka at gcc dot gnu.org

--- Comment #1 from Patrick Palka <ppalka at gcc dot gnu.org> ---
Good point, confirmed.  It seems clear from the definition of the
indirect_strict_weak_order concept that these algorithms are expected to invoke
the predicate as an lvalue, so it's wrong to std::move it.