-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
DO NOT MERGE (yet): Enable -Zdrop-tracking
by default
#97334
Conversation
This comment has been minimized.
This comment has been minimized.
@bors try |
This comment was marked as resolved.
This comment was marked as resolved.
⌛ Trying commit 2b4b7c6512a106b7d25fd906f4c5d9fee6c1a08f with merge 615edd3ad1cf6871c977dc900317cb6c2070fd6b... |
☀️ Try build successful - checks-actions |
@craterbot run mode=build-only |
👌 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
🚧 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
🎉 Experiment
|
2b4b7c6
to
c15747d
Compare
This comment has been minimized.
This comment has been minimized.
c15747d
to
45b65eb
Compare
This comment has been minimized.
This comment has been minimized.
☔ The latest upstream changes (presumably #100654) made this pull request unmergeable. Please resolve the merge conflicts. |
45b65eb
to
e7f6d2b
Compare
This comment has been minimized.
This comment has been minimized.
89ebc24
to
9b8b39d
Compare
This comment has been minimized.
This comment has been minimized.
a576f71
to
bb05901
Compare
@bors try |
⌛ Trying commit bb05901728def04193a6ad433659864581e2cc66 with merge f41a2ab7b0d9462073425dd6c04abbeb4805cb77... |
☀️ Try build successful - checks-actions |
@craterbot run mode=build-only |
👌 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
🚧 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
🎉 Experiment
|
bb05901
to
71e44e8
Compare
The job Click to see the possible cause of the failure (guessed by this bot)
|
Huh, that's weird that we're suddenly getting a bunch of new test failures. I wonder what changed in this code? Maybe #101692 had some effect here? |
#101692 has duplicated each generator test case to have the 3 variants (no drop tracking, drop tracking, and MIR drop tracking). This may be where the errors come from. |
☔ The latest upstream changes (presumably #110252) made this pull request unmergeable. Please resolve the merge conflicts. |
I believe this can be closed now that #107421 has landed? |
Yes! Sorry for the slow response! |
This is not quite ready to merge yet because we still need fixes for a couple of issues (See #97331 for details), but we can probably go ahead and kick off a crater run to see if there are any additional regressions we don't know about yet.
@bors r? @wesleywiser