Skip to content

Fix auto diff failing on inherent impl blocks #140104

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Fix auto diff failing on inherent impl blocks #140104

wants to merge 3 commits into from
+71 −8

Conversation

Shourya742
Copy link
Contributor

closes: #139557

r? @ZuseZ4

@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 Apr 21, 2025
@rustbot
Copy link
Collaborator

rustbot commented Apr 21, 2025

Some changes occurred in compiler/rustc_builtin_macros/src/autodiff.rs

cc @ZuseZ4

@Shourya742 Shourya742 changed the title 2025 04 21 auto diff fails on impl block auto diff fails on impl block Apr 21, 2025
@rust-log-analyzer

This comment has been minimized.

Sign in to view
@fmease fmease changed the title auto diff fails on impl block Fix auto diff failing on inherent impl blocks Apr 21, 2025
@Shourya742 Shourya742 force-pushed the 2025-04-21-auto-diff-fails-on-impl-block branch from c4cbe4f to 11bd84e Compare April 21, 2025 12:04
@ZuseZ4
Copy link
Member

ZuseZ4 commented Apr 21, 2025

Now that we have 3 different pretty tests, can you please create a folder and move them all into tests/pretty/autodiff , to make it easier to run them?
Also, can you please rename the test to something more meaningful, like inherent_impl.rs?
Otherwise lgtm.

@ZuseZ4 ZuseZ4 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-review Status: Awaiting review from the assignee but also interested parties. labels Apr 22, 2025
@Shourya742 Shourya742 force-pushed the 2025-04-21-auto-diff-fails-on-impl-block branch from 11bd84e to b8ca007 Compare April 22, 2025 04:28
@Shourya742
Copy link
Contributor Author

Now that we have 3 different pretty tests, can you please create a folder and move them all into tests/pretty/autodiff , to make it easier to run them? Also, can you please rename the test to something more meaningful, like inherent_impl.rs? Otherwise lgtm.

Added: b8ca007

@Shourya742
Copy link
Contributor Author

@rustbot review

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Apr 22, 2025
@ZuseZ4
Copy link
Member

ZuseZ4 commented Apr 22, 2025

Yep, it passes all tests locally, thanks!
@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Apr 22, 2025

📌 Commit b8ca007 has been approved by ZuseZ4

It is now in the queue for this repository.

@bors 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 Apr 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@ZuseZ4 ZuseZ4

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
No milestone
Development

Successfully merging this pull request may close these issues.

autodiff fails in impl blocks
5 participants
@Shourya742 @rustbot @rust-log-analyzer @ZuseZ4 @bors