Rollup of 10 pull requests #138693

matthiaskrgr · 2025-03-19T07:17:20Z

Successful merges:

clarify BufRead::{fill_buf, consume} docs #136177 (clarify BufRead::{fill_buf, consume} docs)
Remove the regex dependency from coretests #138654 (Remove the regex dependency from coretests)
rustc-dev-guide sync #138655 (rustc-dev-guide sync)
Remove double nesting in post-merge workflow #138656 (Remove double nesting in post-merge workflow)
CI: mirror alpine and centos images to ghcr #138658 (CI: mirror alpine and centos images to ghcr)
coverage: Don't store a body span in FunctionCoverageInfo #138659 (coverage: Don't store a body span in FunctionCoverageInfo)
Revert: Add *_value methods to proc_macro lib #138661 (Revert: Add *_value methods to proc_macro lib)
Remove existing AFIDT implementation #138670 (Remove existing AFIDT implementation)
Various codegen_llvm cleanups #138674 (Various codegen_llvm cleanups)
use then in docs for fuse to enhance readability #138684 (use then in docs for fuse to enhance readability)

r? @ghost
@rustbot modify labels: rollup

Create a similar rollup

Fixes rust-lang#85394

Fix MCP links

…pter-cleaning "Running tests" chapter cleaning

expand ${workspaceFolder} in sample vim config

…eter

Rustc pull

It is only used by a single test, yet would take up unnecessary space once stdlib deps get vendored.

Co-authored-by: Ibraheem Ahmed <ibraheem@ibraheem.ca>

Given that we now only enlarge empty spans to "{" or "}", there shouldn't be any danger of enlarging beyond a function body.

…add_value_retrieval_methods, r=Amanieu" This reverts commit 08dfbf4, reversing changes made to 10bcdad.

clarify BufRead::{fill_buf, consume} docs Fixes rust-lang#85394

matthiaskrgr · 2025-03-19T07:17:41Z

@bors r+ rollup=never p=4

bors · 2025-03-19T07:17:43Z

📌 Commit 3b7faca has been approved by matthiaskrgr

It is now in the queue for this repository.

bors · 2025-03-19T07:17:44Z

🌲 The tree is currently closed for pull requests below priority 100. This pull request will be tested once the tree is reopened.

rust-log-analyzer · 2025-03-19T07:30:32Z

The job x86_64-gnu-llvm-18 failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)

#20 exporting to docker image format
#20 sending tarball 19.5s done
#20 DONE 24.3s
##[endgroup]
Setting extra environment values for docker:  --env ENABLE_GCC_CODEGEN=1 --env GCC_EXEC_PREFIX=/usr/lib/gcc/
[CI_JOB_NAME=x86_64-gnu-llvm-18]
[CI_JOB_NAME=x86_64-gnu-llvm-18]
debug: `DISABLE_CI_RUSTC_IF_INCOMPATIBLE` configured.
---
sccache: Listening on address 127.0.0.1:4226
##[group]Configure the build
configure: processing command line
configure: 
configure: build.configure-args := ['--build=x86_64-unknown-linux-gnu', '--llvm-root=/usr/lib/llvm-18', '--enable-llvm-link-shared', '--set', 'rust.randomize-layout=true', '--set', 'rust.thin-lto-import-instr-limit=10', '--set', 'build.print-step-timings', '--enable-verbose-tests', '--set', 'build.metrics', '--enable-verbose-configure', '--enable-sccache', '--disable-manage-submodules', '--enable-locked-deps', '--enable-cargo-native-static', '--set', 'rust.codegen-units-std=1', '--set', 'dist.compression-profile=balanced', '--dist-compression-formats=xz', '--set', 'rust.lld=false', '--disable-dist-src', '--release-channel=nightly', '--enable-debug-assertions', '--enable-overflow-checks', '--enable-llvm-assertions', '--set', 'rust.verify-llvm-ir', '--set', 'rust.codegen-backends=llvm,cranelift,gcc', '--set', 'llvm.static-libstdcpp', '--enable-new-symbol-mangling']
configure: build.build          := x86_64-unknown-linux-gnu
configure: target.x86_64-unknown-linux-gnu.llvm-config := /usr/lib/llvm-18/bin/llvm-config
configure: llvm.link-shared     := True
configure: rust.randomize-layout := True
configure: rust.thin-lto-import-instr-limit := 10
---
  0     0    0     0    0     0      0      0 --:--:--  0:03:50 --:--:--     0
  0     0    0     0    0     0      0      0 --:--:--  0:03:51 --:--:--     0
  0     0    0     0    0     0      0      0 --:--:--  0:03:51 --:--:--     0
curl: (28) Failed to connect to gcc.gnu.org port 443 after 134682 ms: Couldn't connect to server
error: Cannot download gettext-0.22.tar.gz from http://gcc.gnu.org/pub/gcc/infrastructure/
Command has failed. Rerun with -v to see more details.
Build completed unsuccessfully in 0:08:06
  local time: Wed Mar 19 07:30:22 UTC 2025
  network time: Wed, 19 Mar 2025 07:30:22 GMT
##[error]Process completed with exit code 1.
Post job cleanup.

matthiaskrgr · 2025-03-19T12:38:25Z

@bors treeclosed-

bors · 2025-03-19T12:39:38Z

⌛ Testing commit 3b7faca with merge a7fc463...

bors · 2025-03-19T15:49:38Z

☀️ Test successful - checks-actions
Approved by: matthiaskrgr
Pushing a7fc463 to master...

github-actions · 2025-03-19T15:51:14Z

What is this?

This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.

Comparing c4b38a5 (parent) -> a7fc463 (this PR)

Test differences

Show 431 test diffs

tests::test_unescape_byte_bad (stage 1): pass -> [missing] (J0)
tests::test_unescape_byte_good (stage 1): pass -> [missing] (J0)
tests::test_unescape_byte_str_good (stage 1): pass -> [missing] (J0)
tests::test_unescape_char_bad (stage 1): pass -> [missing] (J0)
tests::test_unescape_char_good (stage 1): pass -> [missing] (J0)
tests::test_unescape_raw_byte_str (stage 1): pass -> [missing] (J0)
tests::test_unescape_raw_str (stage 1): pass -> [missing] (J0)
tests::test_unescape_str_good (stage 1): pass -> [missing] (J0)
tests::test_unescape_str_warn (stage 1): pass -> [missing] (J0)
[crashes] tests/crashes/136286.rs (stage 1): pass -> [missing] (J1)
[crashes] tests/crashes/137706.rs (stage 1): pass -> [missing] (J1)
[crashes] tests/crashes/137895.rs (stage 1): pass -> [missing] (J1)
[ui] tests/ui/feature-gates/literal-escaper.rs (stage 1): pass -> [missing] (J1)
unescape::tests::test_unescape_byte_bad (stage 1): [missing] -> pass (J2)
unescape::tests::test_unescape_byte_good (stage 1): [missing] -> pass (J2)
unescape::tests::test_unescape_byte_str_good (stage 1): [missing] -> pass (J2)
unescape::tests::test_unescape_char_bad (stage 1): [missing] -> pass (J2)
unescape::tests::test_unescape_char_good (stage 1): [missing] -> pass (J2)
unescape::tests::test_unescape_raw_byte_str (stage 1): [missing] -> pass (J2)
unescape::tests::test_unescape_raw_str (stage 1): [missing] -> pass (J2)
unescape::tests::test_unescape_str_good (stage 1): [missing] -> pass (J2)
unescape::tests::test_unescape_str_warn (stage 1): [missing] -> pass (J2)
[crashes] tests/crashes/137706.rs (stage 2): pass -> [missing] (J3)
[crashes] tests/crashes/137706.rs (stage 2): ignore (ignored if rustc wasn't built with debug assertions) -> [missing] (J4)
[crashes] tests/crashes/136286.rs (stage 2): pass -> [missing] (J5)
[crashes] tests/crashes/137895.rs (stage 2): pass -> [missing] (J5)
[ui] tests/ui/feature-gates/literal-escaper.rs (stage 2): pass -> [missing] (J6)

Additionally, 404 doctest diffs were found. These are ignored, as they are noisy.

Job group index

J0: aarch64-apple, aarch64-gnu, arm-android, armhf-gnu, dist-i586-gnu-i586-i686-musl, i686-gnu-1, i686-gnu-2, i686-gnu-nopt-1, i686-gnu-nopt-2, i686-mingw-2, i686-mingw-3, i686-msvc-1, i686-msvc-2, test-various, x86_64-apple-1, x86_64-gnu, x86_64-gnu-llvm-18-1, x86_64-gnu-llvm-18-2, x86_64-gnu-llvm-18-3, x86_64-gnu-llvm-19-1, x86_64-gnu-llvm-19-2, x86_64-gnu-llvm-19-3, x86_64-gnu-nopt, x86_64-gnu-stable, x86_64-mingw-1, x86_64-mingw-2, x86_64-msvc-1, x86_64-msvc-2
J1: x86_64-gnu-llvm-18-3, x86_64-gnu-llvm-19-3
J2: aarch64-apple, aarch64-gnu, i686-gnu-2, i686-gnu-nopt-2, i686-mingw-2, i686-mingw-3, i686-msvc-2, x86_64-apple-1, x86_64-gnu, x86_64-gnu-llvm-18-3, x86_64-gnu-llvm-19-3, x86_64-gnu-nopt, x86_64-gnu-stable, x86_64-mingw-2, x86_64-msvc-2
J3: aarch64-gnu, arm-android, armhf-gnu, i686-gnu-1, i686-gnu-nopt-1, i686-msvc-1, x86_64-gnu, x86_64-gnu-llvm-18-1, x86_64-gnu-llvm-18-2, x86_64-gnu-llvm-19-1, x86_64-gnu-llvm-19-2, x86_64-gnu-nopt, x86_64-gnu-stable, x86_64-mingw-1, x86_64-msvc-1
J4: aarch64-apple, dist-i586-gnu-i586-i686-musl, test-various, x86_64-apple-1
J5: aarch64-apple, aarch64-gnu, arm-android, armhf-gnu, dist-i586-gnu-i586-i686-musl, i686-gnu-1, i686-gnu-nopt-1, i686-msvc-1, test-various, x86_64-apple-1, x86_64-gnu, x86_64-gnu-llvm-18-1, x86_64-gnu-llvm-18-2, x86_64-gnu-llvm-19-1, x86_64-gnu-llvm-19-2, x86_64-gnu-nopt, x86_64-gnu-stable, x86_64-mingw-1, x86_64-msvc-1
J6: aarch64-apple, aarch64-gnu, arm-android, armhf-gnu, dist-i586-gnu-i586-i686-musl, i686-gnu-1, i686-gnu-nopt-1, i686-msvc-1, test-various, x86_64-apple-2, x86_64-gnu, x86_64-gnu-llvm-18-1, x86_64-gnu-llvm-18-2, x86_64-gnu-llvm-19-1, x86_64-gnu-llvm-19-2, x86_64-gnu-nopt, x86_64-gnu-stable, x86_64-mingw-1, x86_64-msvc-1

rust-timer · 2025-03-19T15:52:38Z

📌 Perf builds for each rolled up PR:

PR#	Message	Perf Build Sha
#136177	clarify BufRead::{fill_buf, consume} docs	`d50aa0f2ba6cb6171ebee0b28d81ebcef193deba` (link)
#138654	Remove the regex dependency from coretests	`065ba27de144f340d6d098c554466a8b0f418812` (link)
#138655	rustc-dev-guide sync	`d44a24b923f4867f69c28def78d9031b99330543` (link)
#138656	Remove double nesting in post-merge workflow	`35aae927fe3b287c620b22e28d8566668e141fd6` (link)
#138658	CI: mirror alpine and centos images to ghcr	`d3858baeaa90fcc6a1ef31ecaf6b1219fa72087d` (link)
#138659	coverage: Don't store a body span in `FunctionCoverageInfo`	`d3493e060d195f43ad6367946e61eb2a7fab2bc5` (link)
#138661	Revert: Add *_value methods to proc_macro lib	`67c550f2104c1d57aa535a956023752d20eb9647` (link)
#138670	Remove existing AFIDT implementation	`130a33ab8fd3d91728da6ef74c691ab69c22880e` (link)
#138674	Various codegen_llvm cleanups	`836e0e20769f1aa0cd0b96d8c2844cd59fe2c20c` (link)
#138684	use then in docs for `fuse` to enhance readability	`7bdc6a9f123b177d3f071586a507015a00f39e4e` (link)

previous master: c4b38a5967

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 · 2025-03-19T17:08:36Z

Finished benchmarking commit (a7fc463): comparison URL.

Overall result: ❌ regressions - please read the text below

Our benchmarks found a performance regression caused by this PR.
This might be an actual regression, but it can also be just noise.

Next Steps:

If the regression was expected or you think it can be justified,
please write a comment with sufficient written justification, and add
@rustbot label: +perf-regression-triaged to it, to mark the regression as triaged.
If you think that you know of a way to resolve the regression, try to create
a new PR with a fix for the regression.
If you do not understand the regression or you think that it is just noise,
you can ask the @rust-lang/wg-compiler-performance working group for help (members of this group
were already notified of this PR).

@rustbot label: +perf-regression
cc @rust-lang/wg-compiler-performance

Instruction count

This is the most reliable metric that we have; it was used to determine the overall result at the top of this comment. However, even this metric can sometimes exhibit noise.

	mean	range	count
Regressions ❌ (primary)	6.8%	[0.5%, 14.7%]	10
Regressions ❌ (secondary)	3.2%	[0.2%, 9.5%]	32
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-	-	0
All ❌✅ (primary)	6.8%	[0.5%, 14.7%]	10

Max RSS (memory usage)

Results (primary 3.5%)

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.

	mean	range	count
Regressions ❌ (primary)	3.5%	[2.6%, 4.4%]	2
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-	-	0
All ❌✅ (primary)	3.5%	[2.6%, 4.4%]	2

Cycles

Results (primary 3.2%, secondary 1.9%)

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.

	mean	range	count
Regressions ❌ (primary)	3.2%	[0.8%, 10.9%]	9
Regressions ❌ (secondary)	4.3%	[1.3%, 10.9%]	6
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-2.7%	[-2.9%, -2.5%]	3
All ❌✅ (primary)	3.2%	[0.8%, 10.9%]	9

Binary size

Results (primary 8.3%, secondary 8.8%)

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.

	mean	range	count
Regressions ❌ (primary)	8.3%	[0.9%, 18.1%]	16
Regressions ❌ (secondary)	11.0%	[7.3%, 11.4%]	37
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-0.0%	[-0.1%, -0.0%]	9
All ❌✅ (primary)	8.3%	[0.9%, 18.1%]	16

Bootstrap: 776.901s -> 773.717s (-0.41%)
Artifact size: 365.07 MiB -> 365.52 MiB (0.12%)

lqd · 2025-03-19T17:17:55Z

what.

@rust-timer build 836e0e2

rust-timer · 2025-03-20T01:57:14Z

Finished benchmarking commit (836e0e2): comparison URL.

Overall result: ❌ regressions - please read the text below

Instruction count

This is the most reliable metric that we have; it was used to determine the overall result at the top of this comment. However, even this metric can sometimes exhibit noise.

	mean	range	count
Regressions ❌ (primary)	5.4%	[0.3%, 14.7%]	13
Regressions ❌ (secondary)	3.3%	[0.4%, 9.5%]	31
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-0.1%	[-0.1%, -0.1%]	1
All ❌✅ (primary)	5.4%	[0.3%, 14.7%]	13

Max RSS (memory usage)

Results (primary -0.1%, secondary -2.2%)

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.

	mean	range	count
Regressions ❌ (primary)	2.5%	[2.5%, 2.5%]	1
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-2.7%	[-2.7%, -2.7%]	1
Improvements ✅ (secondary)	-2.2%	[-2.2%, -2.2%]	1
All ❌✅ (primary)	-0.1%	[-2.7%, 2.5%]	2

Cycles

Results (primary 5.5%, secondary 1.0%)

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.

	mean	range	count
Regressions ❌ (primary)	5.5%	[2.1%, 10.3%]	4
Regressions ❌ (secondary)	4.6%	[1.4%, 10.2%]	3
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-2.6%	[-2.8%, -2.5%]	3
All ❌✅ (primary)	5.5%	[2.1%, 10.3%]	4

Binary size

Results (primary 8.4%, secondary 11.1%)

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.

	mean	range	count
Regressions ❌ (primary)	8.4%	[0.9%, 18.0%]	16
Regressions ❌ (secondary)	11.1%	[8.5%, 11.5%]	37
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-	-	0
All ❌✅ (primary)	8.4%	[0.9%, 18.0%]	16

Bootstrap: 776.901s -> 775.979s (-0.12%)
Artifact size: 365.07 MiB -> 365.52 MiB (0.12%)

lqd · 2025-03-20T05:53:19Z

cc @oli-obk there’s probably an unexpected change wrt queries in #138674 causing these regressions

oli-obk · 2025-03-20T06:32:31Z

Huh. That PR was not supposed to change anything

hkBst and others added 30 commits January 28, 2025 10:34

clarify BufRead::{fill_buf, consume} docs

51c5700

Fixes rust-lang#85394

Fix MCP links

2579bb5

Merge pull request rust-lang#2286 from Kobzol/fix-mcp-link

34a340e

Fix MCP links

those should not get shell highlighting

901102f

make 'mdbook test --chapter "Running tests"' pass

35a70bd

add some copy-paste goodness

fdce008

Merge pull request rust-lang#2287 from rust-lang/tshepang-running-cha…

b30d197

…pter-cleaning "Running tests" chapter cleaning

expand ${workspaceFolder} in sample vim config

4aee995

Merge pull request rust-lang#2290 from jyn514/nvim-config

ef46ce7

expand ${workspaceFolder} in sample vim config

Create a safe wrapper around LLVMRustDIBuilderCreateTemplateTypeParam…

eef70a9

…eter

Immediately create an Option instead of reallocating for it later

b4acf7a

Deduplicate template parameter creation

6adc2c1

Create a safe wrapper around LLVMRustDIBuilderCreateSubroutineType

e19e4e3

Create a safe wrapper function around LLVMRustDIBuilderCreateFile

cc41dd4

Create a safe wrapper around LLVMRustDIBuilderCreateBasicType

018032c

Preparing for merge from rustc

6d515a7

Merge from rustc

69ed023

Merge pull request rust-lang#2293 from jieyouxu/rustc-pull

e9d50f4

Rustc pull

Remove double nesting in post-merge workflow

9dac479

Remove the regex dependency from coretests

7f4d3bd

It is only used by a single test, yet would take up unnecessary space once stdlib deps get vendored.

Apply suggestions from code review

e5b86e2

Co-authored-by: Ibraheem Ahmed <ibraheem@ibraheem.ca>

CI: mirror alpine and centos images to ghcr

d76e89f

coverage: Don't refer to the body span when enlarging empty spans

cd2b978

Given that we now only enlarge empty spans to "{" or "}", there shouldn't be any danger of enlarging beyond a function body.

coverage: Don't store a body span in FunctionCoverageInfo

cc8336b

Revert "Rollup merge of rust-lang#136355 - GuillaumeGomez:proc-macro_…

20d04d8

…add_value_retrieval_methods, r=Amanieu" This reverts commit 08dfbf4, reversing changes made to 10bcdad.

Avoid splitting up a layout

1f34b19

Create a safe wrapper around LLVMRustDIBuilderCreateMemberType

f4b0984

Remove existing AFIDT implementation

93b31d9

use then

7a8cdf0

Rollup merge of rust-lang#136177 - hkBst:patch-24, r=ibraheemdev

b52275f

clarify BufRead::{fill_buf, consume} docs Fixes rust-lang#85394

rustbot added T-bootstrap T-compiler T-infra T-libs rollup labels Mar 19, 2025

bors added S-waiting-on-bors and removed S-waiting-on-review labels Mar 19, 2025

bors added the merged-by-bors label Mar 19, 2025

bors merged commit a7fc463 into rust-lang:master Mar 19, 2025
4 of 7 checks passed

rustbot added this to the 1.87.0 milestone Mar 19, 2025

rustbot added the perf-regression label Mar 19, 2025

This comment has been minimized.

Sign in to view

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Rollup of 10 pull requests #138693

Rollup of 10 pull requests #138693

matthiaskrgr commented Mar 19, 2025

matthiaskrgr commented Mar 19, 2025

bors commented Mar 19, 2025

bors commented Mar 19, 2025

rust-log-analyzer commented Mar 19, 2025

matthiaskrgr commented Mar 19, 2025

bors commented Mar 19, 2025

bors commented Mar 19, 2025

github-actions bot commented Mar 19, 2025

rust-timer commented Mar 19, 2025

rust-timer commented Mar 19, 2025

lqd commented Mar 19, 2025

This comment has been minimized.

rust-timer commented Mar 20, 2025

lqd commented Mar 20, 2025 •

edited

Loading

oli-obk commented Mar 20, 2025

Rollup of 10 pull requests #138693

Rollup of 10 pull requests #138693

Conversation

matthiaskrgr commented Mar 19, 2025

matthiaskrgr commented Mar 19, 2025

bors commented Mar 19, 2025

bors commented Mar 19, 2025

rust-log-analyzer commented Mar 19, 2025

matthiaskrgr commented Mar 19, 2025

bors commented Mar 19, 2025

bors commented Mar 19, 2025

github-actions bot commented Mar 19, 2025

Test differences

rust-timer commented Mar 19, 2025

rust-timer commented Mar 19, 2025

Overall result: ❌ regressions - please read the text below

lqd commented Mar 19, 2025

This comment has been minimized.

rust-timer commented Mar 20, 2025

Overall result: ❌ regressions - please read the text below

lqd commented Mar 20, 2025 • edited Loading

oli-obk commented Mar 20, 2025

lqd commented Mar 20, 2025 •

edited

Loading