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 on error propagation with generic #3163

Merged
merged 1 commit into from
Sep 20, 2024

Conversation

Kamiinarii78
Copy link

add a resolve expr visitor on error propagation to avoid internal compiler error when used with generics

gcc/rust/ChangeLog:

* resolve/rust-ast-resolve-expr.cc (ResolveExpr::visit): Add implementation of error propagation visitor
* resolve/rust-ast-resolve-expr.h: Add prototype of error propagation

fixes #3001

add a resolve expr visitor on error propagation to avoid internal
compiler error when used with generics

gcc/rust/ChangeLog:

	* resolve/rust-ast-resolve-expr.cc (ResolveExpr::visit): Add
	implementation of error propagation visitor
	* resolve/rust-ast-resolve-expr.h: Add prototype of error
	propagation

Signed-off-by: benjamin.thos <benjamin.thos@epita.fr>
@philberty philberty added the AST label Sep 16, 2024
@philberty
Copy link
Member

philberty commented Sep 16, 2024

Are you able to add a test for this? Hmm may not just be fiddly to do it you could use the -fcompile-untill flags iirc

@philberty philberty added this pull request to the merge queue Sep 20, 2024
Merged via the queue into Rust-GCC:master with commit 5d703a3 Sep 20, 2024
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Generic arguments are not disambiguated on error propagation expressions
3 participants