-
Notifications
You must be signed in to change notification settings - Fork 13.4k
compiler panic when moving struct definition #93096
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Comments
This should hopefully make it easier to debug incremental compilation bugs like rust-lang#93096 without affecting performance.
@zvkemp Can you try the following to see if you can reproduce this again:
|
@aaron1101 Unfortunately I can't reproduce the issue again using that procedure; in hindsight I should have copied the target directory state before running cargo clean. I'll reply to this issue again if I end up getting it working (or rather not working). |
I think I ran across the same issue, but it was after switching git branches multiple times so finding a reproduction seems hard :( I ran
a clean build resolved it. |
I've had another panic with a similar stacktrace, again after globally renaming a struct. This one is only triggered by @aaron1101 I've backed up the target directory this time, so please let me know if there's anything you'd like me to look at in more detail. Backtrace
|
@zvkemp Are you able to consistently reproduce the crash after running |
I've just had it too, going to try
EDIT: |
…i-obk Show a more informative panic message when `DefPathHash` does not exist This should hopefully make it easier to debug incremental compilation bugs like rust-lang#93096 without affecting performance.
Same thing here, but
But it didn't trigger the error... |
Same here. Running Without thinking twice I ran Anyway, the gist is that I can't provide a minimal source code reproing because 1) it seems unrelated to the specific source code 2) if clean fixes, code probably wouldn't repro anyway. Meta
Error output
Backtrace
|
I have an open source crate having this problem in https://github.com/manishearth/icu4x/tree/ice-dep-hash : I'm unable to reproduce after cargo clean, but the last commit is what caused the ICE |
This is almost certainly a duplicate of this issue and was probably resolved in this PR: |
Code
Code is part of a private repo, so I can't share everything. The change involved moving a struct definition (the
ApplicationSummaryKey
in the below example) from one module to a new submodule of that module. It is triggered bycargo build
orcargo test
(possibly other commands). Was not triggered by nightly, but it's unclear if that's because there wasn't a stale nightly build already.This is the function at the top of the query stack when the panic occurs:
Also note this crate is part of a workspace, and the panic only happens when run from the workspace root; it does not happen if run in the crate's subdirectory. A
cargo clean -p {crate}
fixed the issue, and now I cannot reproduce it.Meta
rustc --version --verbose
:Error output
Backtrace
The text was updated successfully, but these errors were encountered: