A somewhat quiet week. Some PRs had performance runs performed on them, but the changes were merged despite this. Also, we still have issues with certain benchmarks being noisy.
Triage done by @rylev. Revision range: 25a277f03df7e44643ddfcc240d034409cb2f505..cdbe2888979bb8797b05f0d58a6f6e60753983d2
2 Regressions, 2 Improvements, 1 Mixed 0 of them in rollups
Make building THIR a stealable query #85273
- Moderate regression in instruction counts (up to 3.7% on
incr-full
builds oftuple-stress-check
) - Work needed for other work to land. A performance run was performed, but still the change was allowed to land. I've asked why.
Implement the new desugaring from try_trait_v2
#84767
- Moderate regression in instruction counts (up to 2.9% on
full
builds ofcranelift-codegen-check
) - Moderate improvement in instruction counts (up to -1.8% on
full
builds ofdeeply-nested-async-check
) - A much larger performance hit than when the performance suite was run while the pull request was open. This is largely a performance regression with only one non-volatile benchmark showing improvement.
- Biggest change seems to be in the
mir_built
query which seems to make sense given that probably more mir will be produced by this change. - Follow-up happening here.
fix deallocation of immutable allocations #85599
- Moderate improvement in instruction counts (up to -1.9% on
full
builds ofdeeply-nested-async-check
)
std: Attempt again to inline thread-local-init across crates #84876
- Moderate improvement in instruction counts (up to -1.8% on
incr-full
builds ofexterns-debug
) - Moderate regression in instruction counts (up to 1.3% on
full
builds ofregex-opt
)
Avoid zero-length memcpy in formatting #85391
- Moderate regression in instruction counts (up to 3.8% on
incr-patched: println
builds ofcargo-opt
) - Moderate improvement in instruction counts (up to -1.8% on
full
builds ofdeeply-nested-async-debug
) - This seems to have some interactions with LLVM codegen that cause certain benchmarks to fluctuate in performance. These largely cancel each other out.
- Follow-up: #84767.