Reorder early post-inlining passes.

[cjgillot]

RemoveZsts, RemoveUnneededDrops and UninhabitedEnumBranching only depend on types, so they should be executed together early after MIR inlining introduces those types.

This does not change the end-result, but this makes the pipeline a bit more consistent.

[rustbot]

r? @petrochenkov

(rustbot has picked a reviewer for you, use r? to override)

[rustbot]

Some changes occurred to MIR optimizations

cc @rust-lang/wg-mir-opt

[petrochenkov]

r? compiler

[oli-obk]

@bors r+ rollup=never

[bors]

📌 Commit a8c4d43 has been approved by oli-obk

It is now in the queue for this repository.

[bors]

⌛ Testing commit a8c4d43 with merge 6508855...

[bors]

💥 Test timed out

[rust-log-analyzer]

A job failed! Check out the build log: (web)(plain)

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

[cjgillot]

@bors retry

[bors]

⌛ Testing commit a8c4d43 with merge ac177fe...

[rust-log-analyzer]

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

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

[RUSTC-TIMING] rustc_trait_selection test:false 158.169
Compiling rustc_driver_impl v0.0.0 (/checkout/compiler/rustc_driver_impl)
[RUSTC-TIMING] rustc_const_eval test:false 82.578
[RUSTC-TIMING] rustc_middle test:false 241.281
##[error]The runner has received a shutdown signal. This can happen when the runner service is stopped, or a manually started runner is canceled.
##[group]Clock drift check
local time: Thu Jan 11 00:02:11 UTC 2024
local time: Thu Jan 11 00:02:11 UTC 2024
Session terminated, killing shell... ...killed.
##[error]The operation was canceled.
Cleaning up orphan processes

[bors]

💔 Test failed - checks-actions

[cjgillot]

@bors retry

[bors]

⌛ Testing commit a8c4d43 with merge 3a6bf35...

[bors]

☀️ Test successful - checks-actions
Approved by: oli-obk
Pushing 3a6bf35 to master...

[rust-timer]

Finished benchmarking commit (3a6bf35): comparison URL.

Overall result: ❌ regressions - 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.

	mean	range	count
Regressions ❌ (primary)	3.9%	[3.9%, 3.9%]	1
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-	-	0
All ❌✅ (primary)	3.9%	[3.9%, 3.9%]	1

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.

	mean	range	count
Regressions ❌ (primary)	3.4%	[2.0%, 5.8%]	3
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-2.3%	[-3.2%, -1.1%]	3
Improvements ✅ (secondary)	-1.2%	[-1.2%, -1.2%]	1
All ❌✅ (primary)	0.5%	[-3.2%, 5.8%]	6

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.

	mean	range	count
Regressions ❌ (primary)	0.4%	[0.4%, 0.4%]	1
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-0.5%	[-0.5%, -0.5%]	1
Improvements ✅ (secondary)	-	-	0
All ❌✅ (primary)	-0.0%	[-0.5%, 0.4%]	2

Binary size

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

Bootstrap: 668.141s -> 668.115s (-0.00%)
Artifact size: 308.47 MiB -> 308.43 MiB (-0.01%)