Rollup of 3 pull requests#159166
Conversation
This is mostly a cosmetic change. The current form of this function tends to mislead people to assume it is implicitly "correct" in various ways by invoking LLVM as an authority. It is only sort-of correct. More specifically, LLVM mostly doesn't care, and neither do we, except when it comes to FFI. So, we have an arbitrary algorithm that works usefully in most cases. It is not inherently more correct than other arbitrary algorithms, so spell this out and weaken its contract.
…ering, r=jhpratt,Urgau `mut` restriction lowering This PR is part of the ongoing work to implement `mut` restrictions proposed in [RFC 3323](https://rust-lang.github.io/rfcs/3323-restrictions.html), and linked to a [GSoC proposal](https://rust-lang.zulipchat.com/#narrow/channel/421156-gsoc/topic/Project.3A.20Implementing.20impl.20and.20mut.20restrictions/with/592352432). It implements path resolution for `mut` restrictions and lowers the resolved information to HIR. Tracking issue: rust-lang#105077 r? @Urgau cc @jhpratt
…tor-align, r=scottmcm compiler: redescribe llvmlike_vector_align as rust_vector_align This is mostly a cosmetic change. The current form of this function tends to mislead people to assume it is implicitly "correct" in various ways by invoking LLVM as an authority. It is only sort-of correct. More specifically, LLVM mostly doesn't care, and neither do we, except when it comes to FFI. So, we have an arbitrary algorithm that works usefully in most cases. It is not inherently more correct than other arbitrary algorithms, so spell this out and weaken its contract.
…uct, r=joshtriplett
Better comment the struct used by `{read,write}_unaligned`
To address rust-lang#159033 (comment)
r? @RalfJung
|
@bors r+ rollup=never p=5 |
This comment has been minimized.
This comment has been minimized.
|
📌 Perf builds for each rolled up PR:
previous master: be8e82435e 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 be8e824 (parent) -> b69e089 (this PR) Test differencesShow 19 test diffsStage 1
Stage 2
Additionally, 12 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 b69e089eb3ec9976ddec602bf18c3d6a8ce72512 --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 (b69e089): comparison URL. Overall result: ❌ regressions - please read:Our benchmarks found a performance regression caused by this PR. Next Steps:
@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 (secondary -2.4%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary -2.5%, secondary -2.6%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis perf run didn't have relevant results for this metric. Bootstrap: 488.088s -> 489.358s (0.26%) |
Successful merges:
mutrestriction lowering #159125 (mutrestriction lowering){read,write}_unaligned#159157 (Better comment the struct used by{read,write}_unaligned)Failed merges:
r? @ghost
Create a similar rollup