Skip to content

Improve doc in TyKind::CoroutineWitness in compiler - #165219

Open
qouteall wants to merge 1 commit into
rust-lang:mainfrom
qouteall:doc-fix-coroutinewitness
Open

qouteall wants to merge 1 commit into
rust-lang:mainfrom
qouteall:doc-fix-coroutinewitness

Conversation

@qouteall

Copy link
Copy Markdown

Removed outdated "This should only appear as part of the CoroutineArgs."

The "A type representing the types stored inside a coroutine." is not fully accurate because witness type doesn't include upvars, which are also stored in coroutine, so edited.

Added description that it's not a data type, and is used in trait solving.

About LLM: The PR is written manually. The outdated doc was noticed when manually reading code. LLM was used for help understanding the compiler codebase. Changes checked by LLM.

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Oct 31, 2026
@rustbot

rustbot commented Oct 31, 2026

Copy link
Copy Markdown
Collaborator

Thanks for the pull request, and welcome! The Rust Project has assigned @dingxiangfei2009 (or someone else) to review your changes, you should hear from them (or someone else) within the next two weeks.

Please see the contribution instructions and our LLM policy for more information.

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler, types
  • compiler, types expanded to 77 candidates
  • Random selection from 21 candidates

@rust-log-analyzer

This comment has been minimized.

@qouteall
qouteall force-pushed the doc-fix-coroutinewitness branch from 804e0a3 to f838c3d Compare October 31, 2026 05:35
@rust-bors

This comment has been minimized.

@qouteall
qouteall force-pushed the doc-fix-coroutinewitness branch from f838c3d to e11a546 Compare November 1, 2026 01:40
@rustbot

rustbot commented Nov 1, 2026

Copy link
Copy Markdown
Collaborator

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

This branch has not been deployed

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

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants