Skip to content

capture diagnostic output in queries #42513

Closed
@nikomatsakis

Description

@nikomatsakis

Right now, queries produce results, but as a side-effect they may issue diagnostics. For incremental compilation, if we are going to (e.g.) skip typeck, but still produce the same output, we really need to capture this output so that we can replay it.

Right now, to issue a diagnostic, you basically directly invoke methods on the session, such as span_err and friends. When emit() is called, these just directly dump out to stderr.

To fix this issue, we would want to record those diagnostics. There is already a diagnostics object that is created (and thus can in principle be saved) -- but we'd have to find the right place to save it (in the query tables, presumably).

We also have to figure out how to recover the tcx. The easiest way to handle this would be to check the TLS for the current tcx (using ty::tls::with()) and record it that way. We generally discourage using TLS to get the tcx except in specific scenarios, but this may fall under that category.

I'm tagging this for mentoring without instructions. There is still some amount of pending design work here that will require experimentation. Plus, once the work is done, we'll have to figure out (separately) how to actually save the errors to disk and replay them etc.

Metadata

Metadata

Assignees

No one assigned

    Labels

    A-incr-compArea: Incremental compilationC-cleanupCategory: PRs that clean code up or issues documenting cleanup.E-hardCall for participation: Hard difficulty. Experience needed to fix: A lot.E-mentorCall for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions