Skip to content

Top level error handling#121206

Merged
bors merged 9 commits intorust-lang:masterfrom
nnethercote:top-level-error-handling
Feb 22, 2024
Merged

Top level error handling#121206
bors merged 9 commits intorust-lang:masterfrom
nnethercote:top-level-error-handling

Conversation

@nnethercote
Copy link
Contributor

@nnethercote nnethercote commented Feb 16, 2024

The interactions between the following things are surprisingly complicated:

  • emit_stashed_diagnostics,
  • flush_delayed,
  • normal return vs abort_if_errors/FatalError.raise() unwinding in the call to the closure in interface::run_compiler.

This PR disentangles it all.

r? @oli-obk

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-query-system Area: The rustc query system (https://rustc-dev-guide.rust-lang.org/query.html) S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants