-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Implement SpecOptionPartialEq
for cmp::Ordering
#107022
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
r? @m-ou-se (rustbot has picked a reviewer for you, use r? to override) |
This comment was marked as resolved.
This comment was marked as resolved.
417262b
to
3122db7
Compare
It seems worrying that rustc/llvm didn't optimize this.
This PR seems fine for now (r=me), but can you please open an issue (if there isn't one already) and add a comment to |
Noticed as I continue to explore options for having code using
partial_cmp
optimize better.Before:
After:
(Which https://alive2.llvm.org/ce/z/-rop5r says LLVM could just do itself, but there's probably an issue already open for that problem from when this was originally looked at for
Option<NonZeroU8>
and friends.)