use min_specialization for some rustc crates where it requires no changes#72044
use min_specialization for some rustc crates where it requires no changes#72044bors merged 1 commit intorust-lang:masterfrom
Conversation
|
r? @varkor (rust_highfive has picked a reviewer for you, use r? to override) |
src/librustc_metadata/lib.rs
Outdated
There was a problem hiding this comment.
@matthewjasper ICEs shouldn't happen, should they?
error: internal compiler error: src/librustc_trait_selection/traits/specialize/mod.rs:101: When translating substitutions for specialization, the expected specialization failed to hold
thread 'rustc' panicked at 'Box<Any>', src/librustc_errors/lib.rs:904:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
There was a problem hiding this comment.
They shouldn't, but the feature has pretty limited testing.
|
Can you test |
|
Ah I only grepped in |
|
Nope that did not work. And if I mark those as specializable then tons of other impls fail. |
|
@bors r+ rollup |
|
📌 Commit 6f5310bdb207439d8f1ca6f17ed22c158e14339e has been approved by |
|
☔ The latest upstream changes (presumably #72041) made this pull request unmergeable. Please resolve the merge conflicts. |
|
@bors r- |
|
rebased |
|
📌 Commit c82103c has been approved by |
use min_specialization for some rustc crates where it requires no changes and add FIXME for the rest Cc @matthewjasper
use min_specialization for some rustc crates where it requires no changes and add FIXME for the rest Cc @matthewjasper
use min_specialization for some rustc crates where it requires no changes and add FIXME for the rest Cc @matthewjasper
Rollup of 10 pull requests Successful merges: - rust-lang#72014 (Deprecated emoji) - rust-lang#72019 (Fix debug assertion in error code) - rust-lang#72027 (Use CDN for ci-caches on download) - rust-lang#72044 (use min_specialization for some rustc crates where it requires no changes) - rust-lang#72052 (display `ConstKind::Param`) - rust-lang#72067 (Emit a warning when optimization fuel runs out) - rust-lang#72072 (doc: minus (U+2212) instead of dash (U+002D) for negative infinity) - rust-lang#72077 (Improve E0571 wording) - rust-lang#72107 (Clean up E0579 explanation) - rust-lang#72109 (Fix clippy warnings) Failed merges: r? @ghost
and add FIXME for the rest
Cc @matthewjasper