- Notifications
You must be signed in to change notification settings - Fork 14k
Open
Labels
A-mir-optArea: MIR optimizationsArea: MIR optimizationsA-mir-opt-inliningArea: MIR inliningArea: MIR inliningC-bugCategory: This is a bug.Category: This is a bug.P-mediumMedium priorityMedium priorityT-compilerRelevant to the compiler team, which will review and decide on the PR/issue.Relevant to the compiler team, which will review and decide on the PR/issue.regression-from-stable-to-stablePerformance or correctness regression from one stable version to another.Performance or correctness regression from one stable version to another.
Description
Please see #112202 for the PR introducing the regression.
Code
geo v0.22.1 with the following Cargo.lock file. I can try to minify it, but I don't really want to.
Meta
See the bisect below (I originally bisected over a larger range, but I wanted to recreate the message quickly):
~/Downloads/geo-0.22.1$ cargo bisect-rustc --preserve --start 2c718d12599ca00dc3957ec604cb3e87fe437124 --end ffb9b61294b96c389d343a4c55b15400249d74e6 -- build --release bisecting ci builds starting at 2c718d12599ca00dc3957ec604cb3e87fe437124, ending at ffb9b61294b96c389d343a4c55b15400249d74e6 opening existing repository at "rust.git" Found origin remote under name `origin` refreshing repository at "rust.git" fetching (via local git) commits from 2c718d12599ca00dc3957ec604cb3e87fe437124 to ffb9b61294b96c389d343a4c55b15400249d74e6 opening existing repository at "rust.git" Found origin remote under name `origin` refreshing repository at "rust.git" looking up first commit looking up second commit checking that commits are by bors and thus have ci artifacts... finding bors merge commits found 2 bors merge commits in the specified range commit[0] 2023-07-16: Auto merge of #113738 - jyn514:rollup-mjcya4c, r=jyn514 commit[1] 2023-07-16: Auto merge of #113430 - Zalathar:hash, r=b-naber validated commits found, specifying toolchains checking the start range to verify it passes installing 2c718d12599ca00dc3957ec604cb3e87fe437124 testing... RESULT: 2c718d12599ca00dc3957ec604cb3e87fe437124, ===> No checking the end range to verify it does not pass installing ffb9b61294b96c389d343a4c55b15400249d74e6 testing... RESULT: ffb9b61294b96c389d343a4c55b15400249d74e6, ===> Yes searched toolchains 2c718d12599ca00dc3957ec604cb3e87fe437124 through ffb9b61294b96c389d343a4c55b15400249d74e6 checking last toolchain to determine final result installing ffb9b61294b96c389d343a4c55b15400249d74e6 testing... ******************************************************************************** Regression in ffb9b61294b96c389d343a4c55b15400249d74e6 ******************************************************************************** Attempting to search unrolled perf builds ERROR: couldn't find perf build comment Error output
The same error appears without the RUSTFLAGS=-Awarnings it just has a lot of spam.
~/Downloads/geo-0.22.1$ RUSTFLAGS=-Awarnings cargo +1.71.0 build --release Compiling geo v0.22.1 (/home/alex/Downloads/geo-0.22.1) error[E0275]: overflow evaluating the requirement `[closure@src/algorithm/map_coords.rs:888:69: 888:72]: Fn<(geo_types::Coord<T>,)>` | = help: consider increasing the recursion limit by adding a `#![recursion_limit = "256"]` attribute to your crate (`geo`) = note: required for `&[closure@src/algorithm/map_coords.rs:888:69: 888:72]` to implement `Fn<(geo_types::Coord<T>,)>` = note: 128 redundant requirements hidden = note: required for `&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&...` to implement `Fn<(geo_types::Coord<T>,)>` = note: the full type name has been written to '/home/alex/Downloads/geo-0.22.1/target/release/deps/geo-31a6d43c26e8a8cc.long-type-10550508034351353399.txt' For more information about this error, try `rustc --explain E0275`. error: could not compile `geo` (lib) due to previous error Metadata
Metadata
Assignees
Labels
A-mir-optArea: MIR optimizationsArea: MIR optimizationsA-mir-opt-inliningArea: MIR inliningArea: MIR inliningC-bugCategory: This is a bug.Category: This is a bug.P-mediumMedium priorityMedium priorityT-compilerRelevant to the compiler team, which will review and decide on the PR/issue.Relevant to the compiler team, which will review and decide on the PR/issue.regression-from-stable-to-stablePerformance or correctness regression from one stable version to another.Performance or correctness regression from one stable version to another.