Rollup of 4 pull requests#153355
Conversation
llvm-config might output paths that contain spaces, in which case the naive approach of splitting on whitespace breaks; instead we ask llvm-config to quote any paths and use the shell-words crate to parse the output.
Use shell-words to parse output from llvm-config llvm-config might output paths that contain spaces, in which case the naive approach of splitting on whitespace breaks; instead we ask llvm-config to quote any paths and use the [shell-words](https://crates.io/crates/shell-words) crate by @tmiasko (a new dependency) to parse the output. r? ChrisDenton Fixes rust-lang#152707
…dtolnay Stabilize `control_flow_ok` Feature: `control_flow_ok` Tracking issue: rust-lang#140266 r? @dtolnay
…alueFormat-aarch64, r=marcoieni Fix LegacyKeyValueFormat report from docker build: host-aarch64 Part of rust-lang#152305 Following rust-lang#153109 r? @marcoieni
…, r=fmease MGCA: fix type error handling for const array and tuple lowering logic It's a simple fix; just needed to handle Ty::Error in `lower_const_arg_array` and `lower_const_arg_tup` Fixes rust-lang#153254 r? @fmease
|
@bors r+ rollup=never p=5 |
This comment has been minimized.
This comment has been minimized.
…uwer Rollup of 4 pull requests Successful merges: - #152712 (Use shell-words to parse output from llvm-config) - #152911 (Stabilize `control_flow_ok`) - #153223 (Fix LegacyKeyValueFormat report from docker build: host-aarch64) - #153345 (MGCA: fix type error handling for const array and tuple lowering logic)
|
The job Click to see the possible cause of the failure (guessed by this bot) |
|
💔 Test for 9c4bb2e failed: CI. Failed job:
|
|
@bors retry |
…uwer Rollup of 4 pull requests Successful merges: - #152712 (Use shell-words to parse output from llvm-config) - #152911 (Stabilize `control_flow_ok`) - #153223 (Fix LegacyKeyValueFormat report from docker build: host-aarch64) - #153345 (MGCA: fix type error handling for const array and tuple lowering logic)
This comment has been minimized.
This comment has been minimized.
|
💔 Test for 9dca545 failed: CI. Failed job:
|
|
Tree closed for PRs with priority less than 1000. |
|
@bors p=1000 |
This comment has been minimized.
This comment has been minimized.
|
A job failed! Check out the build log: (web) (plain enhanced) (plain) Click to see the possible cause of the failure (guessed by this bot) |
|
@bors treeopen |
|
Tree is now open for merging. |
|
📌 Perf builds for each rolled up PR:
previous master: d2218f5f5c In the case of a perf regression, run the following command for each PR you suspect might be the cause: |
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 d2218f5 (parent) -> d956393 (this PR) Test differencesShow 16 test diffsStage 1
Stage 2
Additionally, 14 doctest diffs were found. These are ignored, as they are noisy. Job group index
Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard d9563937fa3b030c5845811113505070109414d2 --output-dir test-dashboardAnd then open Job duration changes
How to interpret the job duration changes?Job durations can vary a lot, based on the actual runner instance |
|
Finished benchmarking commit (d956393): comparison URL. Overall result: ❌✅ regressions and improvements - no action needed@rustbot label: -perf-regression Instruction countOur most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.
Max RSS (memory usage)Results (primary 2.0%, secondary 5.7%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (secondary 5.9%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 481.344s -> 480.491s (-0.18%) |
Successful merges:
control_flow_ok#152911 (Stabilizecontrol_flow_ok)r? @ghost
Create a similar rollup