Rollup of 9 pull requests#153232
Conversation
…it layout optimization, not just repr(C).
The old URL with `master` resulted in a 404 error - use `main` instead.
- Based on Windows implementation. Just removed support for quote escaping since that is not supported in UEFI. - Tested using OVMF on QEMU Signed-off-by: Ayush Singh <ayush@beagleboard.org>
- Add test for split_paths for UEFI target. - `;` is the path separator. Escaping is not supported. Signed-off-by: Ayush Singh <ayush@beagleboard.org>
These doctests are attached to the `TryFrom` trait. Therefore, it is easier to understand to use the `try_from` method instead of the `try_into` method.
These doctests are attached to the `From` trait. Therefore, it is easier to understand to use the `from` method instead of the `into` method.
* docs: explicitly list env vars checked by temp_dir on Windows On Windows, temp_dir() internally calls GetTempPath2/GetTempPath which checks TMP, TEMP, USERPROFILE environment variables in order. This information was previously only available by following links to Microsoft docs. Making it explicit in Rust's own documentation improves discoverability. Addresses rust-lang#125439. * docs: note env var influence on temp_dir and env_clear on Windows On Windows, �nv::temp_dir() internally calls GetTempPath2/GetTempPath, which checks TMP, TEMP, and USERPROFILE in order. Document this lookup order directly in the emp_dir docs rather than requiring users to follow the link to Microsoft documentation. Also add a note on Command::env_clear explaining that clearing the environment affects the child process's emp_dir(), not the parent's. Closes rust-lang#125439. * docs: drop Windows env_clear temp_dir note
Inhibit all-absent-variant optimization for all enum reprs that inhibit layout optimization, not just repr(C). *[View all comments](https://triagebot.infra.rust-lang.org/gh-comments/rust-lang/rust/pull/146989)* Fixes rust-lang#146984
std: sys: pal: uefi: os: Implement split_paths - Tested using OVMF on QEMU @rustbot label +O-UEFI cc @nicholasbishop
Fix ICE in `try_to_raw_bytes` when array elements have mismatched close: rust-lang#152683 After rust-lang#152001, suffixed integer literals preserve their own type during const lowering, so `try_to_raw_bytes` could call `.to_u8()` on a scalar with size > 1, causing an ICE. Fix by using `try_to_bits(Size::from_bytes(1)).ok()` instead. r? BoxyUwU
std random.rs: update link to RTEMS docs The old URL with `master` resulted in a 404 error - use `main` instead.
…s, r=Mark-Simulacrum docs: note env var influence on `temp_dir` and `env_clear` on Windows On Windows, `env::temp_dir()` internally calls `GetTempPath2`/`GetTempPath`, which checks the `TMP`, `TEMP`, and `USERPROFILE` environment variables in order before falling back to the Windows directory. This lookup order was previously only discoverable by following links to Microsoft documentation. This PR documents the env var lookup order directly in `env::temp_dir` docs and notes `GetTempPath2`'s SYSTEM-identity behavior (`C:\Windows\SystemTemp`). Addresses rust-lang#125439.
cleanup `tests/ui/box`, part 2 Followup to rust-lang#152868. I merged some of the slightly less trivial tests into `basic-operations.rs` and deleted several others entirely. r? @Kivooeo (since you claimed the last one, but feel free to reassign) Part of rust-lang#133895.
… r=joboet style: Update doctests for `TryFrom<integer> for bool` and `From<bool> for float` These doctests are attached to the `TryFrom` trait and the `From` trait. Although `From<U> for T` implies `Into<T> for U` and `TryFrom<U> for T` implies `TryInto<T> for U`, I think it is easier to understand to use the `try_from`/`from` method directly instead of the `try_into`/`into` method.
…jdonszelmann,JonathanBrouwer Fix ICE on empty file with -Zquery-dep-graph added a guard in `DepGraph::register_dep_node_debug_str` so it early-returns when `self.data` is still `None`. this happens for the first few dep-nodes in an empty crate with `-Z query-dep-graph`, which caused the previous `unwrap()` panic. fixes rust-lang#153199
…rror, r=lqd Remove `TranslationError` This function is now infallible, since the diagnostic message and presence of variables is already parsed in the `msg!` or `derive(Diagnostic)` macro. In practice, the result of this function was just unwrapped everywhere anyways... r? @Kivooeo if you feel like reviewing diagnostics PRs again, I have a few more coming :) (Tho other people are also free to steal these PRs)
|
@bors r+ rollup=never p=5 |
|
Trying commonly failed jobs |
This comment has been minimized.
This comment has been minimized.
Rollup of 9 pull requests try-job: test-various try-job: x86_64-gnu-aux try-job: x86_64-gnu-llvm-21-3 try-job: x86_64-msvc-1 try-job: aarch64-apple try-job: x86_64-mingw-1
This comment has been minimized.
This comment has been minimized.
|
📌 Perf builds for each rolled up PR:
previous master: 38c0de8dcb 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 38c0de8 (parent) -> 8215374 (this PR) Test differencesShow 65 test diffsStage 1
Stage 2
Additionally, 26 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 821537498d4f05f948fe4406aad99742f0e07618 --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 (8215374): comparison URL. Overall result: ❌ regressions - 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)This benchmark run did not return any relevant results for this metric. CyclesThis benchmark run did not return any relevant results for this metric. Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 481.431s -> 479.99s (-0.30%) |
Successful merges:
try_to_raw_byteswhen array elements have mismatched #152794 (Fix ICE intry_to_raw_byteswhen array elements have mismatched)temp_dirandenv_clearon Windows #153054 (docs: note env var influence ontemp_dirandenv_clearon Windows)tests/ui/box, part 2 #153061 (cleanuptests/ui/box, part 2)TryFrom<integer> for boolandFrom<bool> for float#153197 (style: Update doctests forTryFrom<integer> for boolandFrom<bool> for float)TranslationError#153228 (RemoveTranslationError)r? @ghost
Create a similar rollup