3

Add VarDebugInfo to Stable MIR by ouz-a · Pull Request #117972 · rust-lang/rust...

 9 months ago
source link: https://github.com/rust-lang/rust/pull/117972
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.
neoserver,ios ssh client

Add VarDebugInfo to Stable MIR #117972

Conversation

Contributor

@ouz-a ouz-a

commented

Nov 16, 2023

•

edited

Previously we omitted VarDebugInfo because we didn't have Projection now that #117517 is merged it's possible to add VarDebugInfo information in Body. This PR adds stable version of the VarDebugInfo to Body

r? @celinval

Collaborator

Failed to set assignee to celinval: invalid assignee

Note: Only org members with at least the repository "read" role, users with write permissions, or people who have commented on the PR may be assigned.

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

Nov 16, 2023

Collaborator

Contributor

r? celinval

I created rust-lang/team#1119 to hopefully fix the assignment.

Contributor

☔ The latest upstream changes (presumably #118023) made this pull request unmergeable. Please resolve the merge conflicts.

Contributor

@bors r+ rollup

Awesome! Thanks

Contributor

Author

I don't think that worked 😅

Contributor

@bors r+ rollup

Contributor

📌 Commit 965f46b has been approved by celinval

It is now in the queue for this repository.

bors

added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

labels

Nov 20, 2023

Contributor

☔ The latest upstream changes (presumably #118015) made this pull request unmergeable. Please resolve the merge conflicts.

bors

added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author.

and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

labels

Nov 21, 2023

Contributor

@bors r+ rollup

Contributor

📌 Commit d0dd19a has been approved by celinval

It is now in the queue for this repository.

bors

added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author.

labels

Nov 21, 2023

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

Reviewers

Back to tour

celinval

celinval left review comments
Assignees

celinval

Labels
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.
Projects

None yet

Milestone

1.76.0

Development

Successfully merging this pull request may close these issues.

None yet

4 participants

report

Recommend

About Joyk


Aggregate valuable and interesting links.
Joyk means Joy of geeK