You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@frankmcsherry reported a big runtime improvement by switching from .merge(j) to .merge_by(j, Ord::cmp), with merge::wrapper showing up in the profile.
.partial_ord() call being harmful?
function pointer vs inlined closure issue?
(Profile says: It's both)
The text was updated successfully, but these errors were encountered:
Recommending calling mode .merge_by(j, |a, b| if a <= b { Ordering::Less } else { Ordering::Greater }) as it is now (this is the best way for integers)..
@frankmcsherry reported a big runtime improvement by switching from
.merge(j)
to.merge_by(j, Ord::cmp)
, withmerge::wrapper
showing up in the profile..partial_ord()
call being harmful?(Profile says: It's both)
The text was updated successfully, but these errors were encountered: