Skip to content
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.

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

Fix test for issue 80706 #103635

Closed
wants to merge 1 commit into from
Closed

Fix test for issue 80706 #103635

wants to merge 1 commit into from

Conversation

Rageking8
Copy link
Contributor

Fixes #80706

Actually afaict the test is already good (i.e. the check-pass had already long been changed to build-pass to catch the ICE during codegen (flagged out by tmandry)). So this PR just cleans the test up. Correct me if I am wrong. cc @compiler-errors since you last edited this file.

r? @lcnr

@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 27, 2022
@compiler-errors
Copy link
Member

What's exactly going on here? All of these changes look superficial except for removing // edition:2018, which is going to make the test fail for an unrelated reason.

@Rageking8
Copy link
Contributor Author

What's exactly going on here? All of these changes look superficial except for removing // edition:2018, which is going to make the test fail for an unrelated reason.

I removed that erroneous change, since the remaining changes are pretty superficial we can just go with it or just close this PR and the issue above, since I believe the issue is fixed. Compiler-errors, your call.

@Rageking8
Copy link
Contributor Author

r? @compiler-errors

@rustbot rustbot assigned compiler-errors and unassigned lcnr Oct 27, 2022
@compiler-errors
Copy link
Member

The file is syntactically equivalent except for some formatting and ordering of items, so this PR doesn't do anything as far as I can tell. Closing.

@Rageking8 Rageking8 deleted the fix-test-for-issue-80706 branch October 27, 2022 16:00
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
4 participants