Skip to content

Rollup of 4 pull requests - #114272

Merged
bors merged 9 commits into
rust-lang:masterfrom
workingjubilee:rollup-ll9lwon
Jul 31, 2023
Merged

Rollup of 4 pull requests#114272
bors merged 9 commits into
rust-lang:masterfrom
workingjubilee:rollup-ll9lwon

Conversation

@workingjubilee

Copy link
Copy Markdown
Member

Successful merges:

r? @ghost
@rustbot modify labels: rollup

Create a similar rollup

joboetand others added 9 commits April 27, 2023 09:32
Bump its stabilization version several times along
the way to accommodate changes in release processes.
Co-authored-by: Mara Bos <m-ou.se@m-ou.se>
Co-authored-by: Trevor Gross <t.gross35@gmail.com>
Stabilize const-weak-new
This is a fairly uncontroversial library stabilization, so I'm going ahead and proposing it to ride the trains to stable.
This stabilizes the following APIs, which are defined to be non-allocating constructors.
```rust
// alloc::rc
impl<T> Weak<T> {
pub const fn new() -> Weak<T>;
}
// alloc::sync
impl<T> Weak<T> {
pub const fn new() -> Weak<T>;
}
```
Closesrust-lang#95091
``@rustbot`` modify labels: +needs-fcp
…ngjubilee
Use `LazyLock` to lazily resolve backtraces
By using TAIT to name the initializing closure, `LazyLock` can be used to replace the current `LazilyResolvedCapture`.
…-missing-opaque, r=spastorino
Don't install default projection bound for return-position `impl Trait` in trait methods with no body
This ensures that we never try to project to an opaque type in a trait method that has no body to infer its hidden type, which means we never later call `type_of` on that opaque. This is because opaque types try to reveal their hidden type when proving auto traits.
I thought about this a lot, and I think this is a fix that's less likely to introduce other strange downstream ICEs than rust-lang#113461.
Fixesrust-lang#113434
r? `@spastorino`
Fix empty_write since rust version attribute
Fixup of rust-lang#98154 for the rust version.
r? ``@workingjubilee``
@rustbotrustbot 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. T-libs Relevant to the library team, which will review and decide on the PR/issue. rollup A PR which is a rollup labels Jul 31, 2023
@workingjubilee

Copy link
Copy Markdown
MemberAuthor

@bors r+ rollup=never p=4

@bors

bors commented Jul 31, 2023

Copy link
Copy Markdown
Collaborator

📌 Commit 495e657 has been approved by workingjubilee

It is now in the queue for this repository.

@borsbors 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 Jul 31, 2023
@bors

bors commented Jul 31, 2023

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 495e657 with merge dfc9d3f...

@bors

bors commented Jul 31, 2023

Copy link
Copy Markdown
Collaborator

☀️ Test successful - checks-actions
Approved by: workingjubilee
Pushing dfc9d3f to master...

@borsbors added the merged-by-bors This PR was explicitly merged by bors. label Jul 31, 2023
@bors
bors merged commit dfc9d3f into rust-lang:masterJul 31, 2023
@rustbotrustbot added this to the 1.73.0 milestone Jul 31, 2023
@rust-timer

Copy link
Copy Markdown
Collaborator

📌 Perf builds for each rolled up PR:

PR#MessagePerf Build Sha
#95965Stabilize const-weak-newaaa58781597aa9d34d321b491168ee7e510348ea (link)
#109075Use LazyLock to lazily resolve backtracescda7b6d23b278697bf0cb457e00d594dd4a8a33b (link)
#113741Don't install default projection bound for return-position …893dbc50d829bd01dee445f551cdbd4bda740cab (link)
#114268Fix empty_write since rust version attributeff60a8b05851f6937f2fec1e5fc0feeb856e56a7 (link)

previous master: d9feb02e8d

In the case of a perf regression, run the following command for each PR you suspect might be the cause: @rust-timer build $SHA

@rust-timer

Copy link
Copy Markdown
Collaborator

Finished benchmarking commit (dfc9d3f): comparison URL.

Overall result: ✅ improvements - no action needed

@rustbot label: -perf-regression

Instruction count

This is a highly reliable metric that was used to determine the overall result at the top of this comment.

meanrangecount
Regressions ❌
(primary)
--0
Regressions ❌
(secondary)
--0
Improvements ✅
(primary)
--0
Improvements ✅
(secondary)
-5.7%[-5.7%, -5.7%]1
All ❌✅ (primary)--0

Max RSS (memory usage)

Results

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

meanrangecount
Regressions ❌
(primary)
--0
Regressions ❌
(secondary)
--0
Improvements ✅
(primary)
-6.4%[-6.4%, -6.4%]1
Improvements ✅
(secondary)
--0
All ❌✅ (primary)-6.4%[-6.4%, -6.4%]1

Cycles

Results

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

meanrangecount
Regressions ❌
(primary)
--0
Regressions ❌
(secondary)
2.8%[2.8%, 2.8%]1
Improvements ✅
(primary)
--0
Improvements ✅
(secondary)
-3.3%[-3.3%, -3.3%]1
All ❌✅ (primary)--0

Binary size

This benchmark run did not return any relevant results for this metric.

Bootstrap: 652.627s -> 652.418s (-0.03%)

@workingjubilee
workingjubilee deleted the rollup-ll9lwon branch July 31, 2023 17:51
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged-by-borsThis PR was explicitly merged by bors.rollupA PR which is a rollupS-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.T-libsRelevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@workingjubilee@bors@rust-timer@rustbot@joboet@compiler-errors@SUPERCILEX@CAD97