Update hashbrown to 0.17#155154
Conversation
|
These commits modify the If this was unintentional then you should revert the changes before this PR is merged. |
|
rustbot has assigned @Mark-Simulacrum. Use Why was this reviewer chosen?The reviewer was selected based on:
|
|
@bors try |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Update hashbrown to 0.17
|
@bors retry The runner looks to have hung -- looking in codebuild, it's been mostly idle for hours (though using ~91% of memory on the instance). Retrying to hopefully get a successful try build. |
|
❗ You can only retry pull requests that are approved and have a previously failed auto build. |
|
@bors try cancel |
|
Try build cancelled. Cancelled workflows: |
|
@bors try @rust-timer queue |
This comment has been minimized.
This comment has been minimized.
Update hashbrown to 0.17
This comment has been minimized.
This comment has been minimized.
|
Finished benchmarking commit (c349988): comparison URL. Overall result: ✅ improvements - no action neededBenchmarking means the PR may be perf-sensitive. It's automatically marked not fit for rolling up. Overriding is possible but disadvised: it risks changing compiler perf. @bors rollup=never 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 0.7%, secondary -4.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary -4.7%, secondary 1.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (primary -0.8%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 490.796s -> 489.879s (-0.19%) |
|
@bors r+ |
|
Just FYI, if you run a second try build, it will automatically cancel the previous one :) |
This comment has been minimized.
This comment has been minimized.
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 540f43a (parent) -> 532f8f1 (this PR) Test differencesNo test diffs found Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard 532f8f1443be8f605bcc9da7968e3476d3f27d26 --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 (532f8f1): 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 1.7%, secondary -1.2%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (secondary -6.8%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (primary -0.2%, secondary -0.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 491.503s -> 490.376s (-0.23%) |
See library's rust-lang#155154 for the bug fixes this brings. This PR also updates `indexmap` in the compiler as a direct dependent.
compiler: update hashbrown to 0.17 See library's #155154 for the bug fixes this brings. This PR also updates `indexmap` in the compiler as a direct dependent.
compiler: update hashbrown to 0.17 See library's #155154 for the bug fixes this brings. This PR also updates `indexmap` in the compiler as a direct dependent.
compiler: update hashbrown to 0.17 See library's rust-lang/rust#155154 for the bug fixes this brings. This PR also updates `indexmap` in the compiler as a direct dependent.
compiler: update hashbrown to 0.17 See library's rust-lang/rust#155154 for the bug fixes this brings. This PR also updates `indexmap` in the compiler as a direct dependent.
View all comments
The main benefit of this update is to include one potential UB fix and one bug; relevant changelog entries:
RawTableInner::fallible_with_capacity(Fix potential UB inRawTableInner::fallible_with_capacityhashbrown#692)cc @Amanieu
Also cc @RalfJung who had also noticed the UB issue with
-Zmiri-recursive-validation.