-
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
Make RawVec private to alloc #90542
Make RawVec private to alloc #90542
Conversation
RawVec was previously exposed for compiler-internal use (libarena specifically) in 1acbb0a Since it is unstable, doc-hidden and has no associated tracking issue it was never meant for public use. And since it is no longer used outside alloc itself it can be made private again. Also remove some functions that are dead due to lack of internal users.
(rust-highfive has picked a reviewer for you, use r? to override) |
Previous zulip discussion. https://rust-lang.zulipchat.com/#narrow/stream/219381-t-libs/topic/Using.20RawVec.20directly @joshtriplett suggested doing a crater run before merging. |
@bors try |
⌛ Trying commit 7afe6f5 with merge e36c2954fdc132a34575813e6386ff9dc73f6bca... |
☀️ Try build successful - checks-actions |
r=me if this passes crater. Thanks for working on this! |
Oh yeah, I have crater permissions now. 🦹 @craterbot check |
@craterbot ping |
🏓 Pong! ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
@craterbot check |
🚨 Error: missing start toolchain 🆘 If you have any trouble with Crater please ping |
@craterbot check start=stable |
🚨 Error: missing end toolchain 🆘 If you have any trouble with Crater please ping |
@craterbot check start=stable end=e36c2954fdc132a34575813e6386ff9dc73f6bca |
👌 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 🆘 Can someone from the infra team check in on this? @rust-lang/infra |
2 similar comments
🚨 Experiment 🆘 Can someone from the infra team check in on this? @rust-lang/infra |
🚨 Experiment 🆘 Can someone from the infra team check in on this? @rust-lang/infra |
@craterbot retry |
🛠️ 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 🆘 Can someone from the infra team check in on this? @rust-lang/infra |
1 similar comment
🚨 Experiment 🆘 Can someone from the infra team check in on this? @rust-lang/infra |
⌛ Testing commit 7afe6f5 with merge 61090d2677b1828e0843620d7a67e81fe30972fc... |
The job Click to see the possible cause of the failure (guessed by this bot)
|
💔 Test failed - checks-actions |
https://github.com/rust-lang-ci/rust/runs/4195468557?check_suite_focus=true#step:25:13163
again? |
⌛ Testing commit 7afe6f5 with merge db2642f3e18ef65e52ecd7fb784372d0516fe550... |
💔 Test failed - checks-actions |
The job Click to see the possible cause of the failure (guessed by this bot)
|
This PR is cursed @bors retry |
☀️ Test successful - checks-actions |
Finished benchmarking commit (1b12d01): comparison url. Summary: This change led to very large relevant mixed results 🤷 in compiler performance.
If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf. Next Steps: If you can justify the regressions found in this perf run, please indicate this with @rustbot label: +perf-regression |
RawVec was previously exposed for compiler-internal use (libarena specifically) in 1acbb0a
Since it is unstable, doc-hidden and has no associated tracking issue it was never meant for public use. And since
it is no longer used outside alloc itself it can be made private again.
Also remove some functions that are dead due to lack of internal users.