-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
Implement SpecOptionPartialEq
for cmp::Ordering
#107022
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 |
…iaskrgr Rollup of 8 pull requests Successful merges: - rust-lang#107022 (Implement `SpecOptionPartialEq` for `cmp::Ordering`) - rust-lang#107100 (Use proper `InferCtxt` when probing for associated types in astconv) - rust-lang#107103 (Use new solver in `evaluate_obligation` query (when new solver is enabled)) - rust-lang#107190 (Recover from more const arguments that are not wrapped in curly braces) - rust-lang#107306 (Correct suggestions for closure arguments that need a borrow) - rust-lang#107339 (internally change regions to be covariant) - rust-lang#107344 (Minor tweaks in the new solver) - rust-lang#107373 (Don't merge vtables when full debuginfo is enabled.) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
Remove an unneeded helper from the tuple library code Thanks to rust-lang#107022, this is just what `==` does, so we don't need the helper here anymore.
…riplett Remove an unneeded helper from the tuple library code Thanks to rust-lang#107022, this is just what `==` does, so we don't need the helper here anymore.
Rollup merge of rust-lang#118307 - scottmcm:tuple-eq-simpler, r=joshtriplett Remove an unneeded helper from the tuple library code Thanks to rust-lang#107022, this is just what `==` does, so we don't need the helper here anymore.
Remove an unneeded helper from the tuple library code Thanks to rust-lang/rust#107022, this is just what `==` does, so we don't need the helper here anymore.
Remove an unneeded helper from the tuple library code Thanks to rust-lang/rust#107022, this is just what `==` does, so we don't need the helper here anymore.
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.)