Repository navigation
Conversation
This comment has been minimized.
This comment has been minimized.
|
Some changes occurred to the intrinsics. Make sure the CTFE / Miri interpreter cc @rust-lang/miri, @RalfJung, @oli-obk, @lcnr
cc @rust-lang/wg-const-eval
cc @rust-lang/wg-const-eval |
|
#147368 only merged a few days ago, those are trivial but we should still allow them to soak for at least a release cycle. |
| /// Otherwise it's immediate UB. | ||
| #[rustc_const_unstable(feature = "disjoint_bitor", issue = "135758")] | ||
| #[rustc_intrinsic_const_stable_indirect] | ||
| #[rustc_allow_const_fn_unstable(const_trait_impl, core_intrinsics_fallbacks)] |
There was a problem hiding this comment.
This is not great, it means we are not checking that the fallback impl actually only needs stable const stuff. Can we do without core_intrinsics_fallbacks?
| #[rustc_allow_const_fn_unstable(const_trait_impl, core_intrinsics_fallbacks)] | |
| #[rustc_allow_const_fn_unstable(const_trait_impl)] |
There was a problem hiding this comment.
Okay currently this does not work.
I am not comfortable stabilizing more intrinsics with trait-based fallback impls when we cannot actually do const stability checks in those fallback impls. So this is blocked on supporting #[rustc_const_stable_indirect] on traits.
There was a problem hiding this comment.
cc discussion on funnel shifts: #161015 (comment)
There was a problem hiding this comment.
To avoid blocking on const stability checking improvements, I'd suggest to make this PR only stabilize unchecked_disjoint_bitor for non-const callers.
Closes #135758
Public API:
Implementation prs: #135760, #147368
Could we get a FCP started for this ^^?
r? @clarfonthey (since you've expressed interest in stabilization)