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

Fix ICE when emitting an error during cfg strip #2639

Merged
merged 1 commit into from
Sep 27, 2023

Conversation

P-E-P
Copy link
Member

@P-E-P P-E-P commented Sep 26, 2023

When an error was emitted during the cfg strip pass by the crate loader, it was ignored and the error state propagated until another pass (name resolver).

Fixes #2638

When an error was emitted during the cfg strip pass by the crate loader,
it was ignored and the error state propagated until another pass
(name resolver).

gcc/rust/ChangeLog:

	* rust-session-manager.cc (Session::expansion): Add early break on
	error.

Signed-off-by: Pierre-Emmanuel Patry <[email protected]>
@P-E-P P-E-P added this to the GCC 14 Stage 3 milestone Sep 26, 2023
@P-E-P P-E-P requested a review from CohenArthur September 26, 2023 12:59
@P-E-P P-E-P self-assigned this Sep 26, 2023
@P-E-P P-E-P marked this pull request as ready for review September 26, 2023 13:07
@P-E-P P-E-P added this pull request to the merge queue Sep 27, 2023
Merged via the queue into Rust-GCC:master with commit 2897221 Sep 27, 2023
9 checks passed
Copy link
Member

@CohenArthur CohenArthur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good! thanks for fixing and merging this :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

ICE on crate not found
2 participants