-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Further unify validity intrinsics #108505
Further unify validity intrinsics #108505
Conversation
(rustbot has picked a reviewer for you, use r? to override) |
Some changes occurred in compiler/rustc_codegen_cranelift cc @bjorn3 Some changes occurred to the CTFE / Miri engine cc @rust-lang/miri Some changes occurred to MIR optimizations cc @rust-lang/wg-mir-opt |
Thanks for the PR, @Nilstrieb! I'll take a closer look shortly. |
☔ The latest upstream changes (presumably #108521) made this pull request unmergeable. Please resolve the merge conflicts. |
Also merges the inhabitedness check into the query to further unify the code paths.
bfcc46c
to
5f593da
Compare
Looks good to me -- much more unified indeed 🙂 |
…iaskrgr Rollup of 10 pull requests Successful merges: - rust-lang#108376 (compiler/rustc_session: fix sysroot detection logic) - rust-lang#108400 (add llvm cgu instructions stats to perf) - rust-lang#108496 (fix rust-lang#108495, postfix decrement and prefix decrement has no warning) - rust-lang#108505 (Further unify validity intrinsics) - rust-lang#108520 (Small cleanup to `one_bound_for_assoc_type`) - rust-lang#108560 (Some `infer/mod.rs` cleanups) - rust-lang#108563 (Make mailmap more correct) - rust-lang#108564 (Fix `x clean` with specific paths) - rust-lang#108571 (Add contains_key to SortedIndexMultiMap) - rust-lang#108578 (Update Fuchsia platform team members) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
Also merges the inhabitedness check into the query to further unify the
code paths.
Depends on #108364