-
Notifications
You must be signed in to change notification settings - Fork 12.6k
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
Compile time regression with _large_ number of slices #48009
Labels
C-enhancement
Category: An issue proposing an enhancement or a PR with one.
E-needs-test
Call for participation: An issue has been fixed and does not reproduce, but no test has been added.
I-compiletime
Issue: Problems and improvements with respect to compile times.
T-compiler
Relevant to the compiler team, which will review and decide on the PR/issue.
Comments
TimNN
added
C-enhancement
Category: An issue proposing an enhancement or a PR with one.
I-compiletime
Issue: Problems and improvements with respect to compile times.
labels
Feb 6, 2018
Seems this is still a thing even with #46882. It chokes by turning it into an error:
|
This... unregressed? Time Passes
|
jonas-schievink
added
T-compiler
Relevant to the compiler team, which will review and decide on the PR/issue.
E-needs-test
Call for participation: An issue has been fixed and does not reproduce, but no test has been added.
labels
Jan 31, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
C-enhancement
Category: An issue proposing an enhancement or a PR with one.
E-needs-test
Call for participation: An issue has been fixed and does not reproduce, but no test has been added.
I-compiletime
Issue: Problems and improvements with respect to compile times.
T-compiler
Relevant to the compiler team, which will review and decide on the PR/issue.
cc open-i18n/rust-unic#199
The simplest way to show this is to use
unic_ucd_name
0.6.0, which has two huge files for the Unicode code point Name property,name_map.rsv
(1.56 MB of('character', &[PARTS, OF, ITS, NAME]),
) andname_values.rsd
(444 KB ofconst NAME_BIT: &str = "NAME_BIT";
). The purpose is to generate a large binary-search slice from characters to their name while deduplicating very common name fragments likeLETTER
.With this small
main.rs
and a non-existent
target
folder, I get the following compile times:time-passes
forunic_ucd_name
(
x86_64-pc-windows-msvc
)The text was updated successfully, but these errors were encountered: