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

rustc lost compiler flag information in its backtrace #101588

Closed
matthiaskrgr opened this issue Sep 8, 2022 · 2 comments · Fixed by #101595
Closed

rustc lost compiler flag information in its backtrace #101588

matthiaskrgr opened this issue Sep 8, 2022 · 2 comments · Fixed by #101595
Assignees
Labels
C-bug Category: This is a bug. P-medium Medium priority regression-untriaged Untriaged performance or correctness regression.

Comments

@matthiaskrgr
Copy link
Member

Noticed while digging around in incremental compilation bugs:
I run something like rustc 101518-1.rs -Cincremental=/tmp/a -Zincremental-verify-ich=yes --crate-type lib

rust 1.58.0 would print the flags that caused the ICE like this:

error: internal compiler error: unexpected panic

note: the compiler unexpectedly panicked. this is a bug.

note: we would appreciate a bug report: https://github.com/rust-lang/rust/issues/new?labels=C-bug%2C+I-ICE%2C+T-compiler&template=ice.md

note: rustc 1.58.0 (02072b482 2022-01-11) running on x86_64-unknown-linux-gnu

note: compiler flags: -Z incremental-verify-ich=yes -C incremental --crate-type lib

while 1.60.0 would print it like this:

error: internal compiler error: unexpected panic

note: the compiler unexpectedly panicked. this is a bug.

note: we would appreciate a bug report: https://github.com/rust-lang/rust/issues/new?labels=C-bug%2C+I-ICE%2C+T-compiler&template=ice.md

note: rustc 1.60.0 (7737e0b5c 2022-04-04) running on x86_64-unknown-linux-gnu

note: compiler flags: -C incremental -Z incremental --crate-type lib

The problem here is that -Z incremental-verify-ich=yes somehow became -Z incremental between 1.58 and 1.60, and -Zincremental is not even a valid flag 😅

The current rustc 1.65.0-nightly (c2804e6ec 2022-09-07) output is note: compiler flags: -C incremental -Z incremental --crate-type lib which is quite garbage since both, the actual -Cincremental path and the rest of -Zincremental are missing,

@matthiaskrgr matthiaskrgr added C-bug Category: This is a bug. regression-untriaged Untriaged performance or correctness regression. labels Sep 8, 2022
@rustbot rustbot added the I-prioritize Issue: Indicates that prioritization has been requested for this issue. label Sep 8, 2022
@ehuss
Copy link
Contributor

ehuss commented Sep 8, 2022

Apologies, I accidentally broke this in #92310. I'll post a fix.

@ehuss ehuss self-assigned this Sep 8, 2022
@bors bors closed this as completed in 3ddb048 Sep 10, 2022
@apiraino
Copy link
Contributor

WG-prioritization assigning priority (Zulip discussion).

@rustbot label -I-prioritize +P-medium

@rustbot rustbot added P-medium Medium priority and removed I-prioritize Issue: Indicates that prioritization has been requested for this issue. labels Oct 20, 2022
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. P-medium Medium priority regression-untriaged Untriaged performance or correctness regression.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants