Skip to content
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

ICE during build of curve25519-dalek - index out of bounds #121139

Closed
maxfunke opened this issue Feb 15, 2024 · 2 comments
Closed

ICE during build of curve25519-dalek - index out of bounds #121139

maxfunke opened this issue Feb 15, 2024 · 2 comments
Assignees
Labels
C-bug Category: This is a bug. I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ S-needs-repro Status: This issue has no reproduction and needs a reproduction to make progress. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@maxfunke
Copy link

Log in CI while cargo build

Meta

rustc 1.75.0 (82e1608df 2023-12-21) running on x86_64-pc-windows-msvc

Error output

thread 'rustc' panicked at compiler\rustc_metadata\src\rmeta\decoder.rs:962:10:
index out of bounds: the len is 0 but the index is 0

note: rustc 1.75.0 (82e1608df 2023-12-21) running on x86_64-pc-windows-msvc
note: compiler flags: --crate-type rlib -C opt-level=3 -C embed-bitcode=no
note: some of the compiler flags provided by cargo are hidden
query stack during panic:
#0 [resolver_for_lowering] getting the resolver for lowering
end of query stack
   Compiling darling_core v0.20.3
error: could not compile `curve25519-dalek` (lib)

The error is not reproducable though.

@maxfunke maxfunke added C-bug Category: This is a bug. I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Feb 15, 2024
@rustbot rustbot added the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label Feb 15, 2024
@jieyouxu jieyouxu added A-incr-comp Area: Incremental compilation S-needs-repro Status: This issue has no reproduction and needs a reproduction to make progress. and removed needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. labels Feb 15, 2024
@saethlin saethlin removed the A-incr-comp Area: Incremental compilation label Mar 9, 2024
@saethlin saethlin self-assigned this Mar 9, 2024
@saethlin
Copy link
Member

saethlin commented Mar 9, 2024

I suspect this is another out-of-disk crash, that should hopefully be fixed by all my PRs, the last of which ships in 1.77. It's definitely not an incremental compilation issue.

@saethlin
Copy link
Member

I think the root cause of this issue was rustc incorrectly handling/reporting I/O errors. I've landed a number of PRs to address the problem; the most recent of those is #119510 which is now part of the latest stable toolchain, 1.77. Therefore, even though I have not reproduced exactly what you've reported here, I'm going to close this because I am reasonably confident that this bug is now fixed on stable.

Of course it is possible that I am wrong and the bug you've reported here is not fixed. If you happen to run into problems like this in 1.77 or later please do not hesitate to open a new issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug. I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ S-needs-repro Status: This issue has no reproduction and needs a reproduction to make progress. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

4 participants