Rollup of 7 pull requests#155710
Conversation
Commit 623c7d7 accidentally changed the return value from REGULAR_UNION to RegularEnum when converting string literals to enum values. Commit b17670d then renamed RegularUnion to Union, but the buggy return statement remained unchanged. This caused unions to be misclassified as enums, preventing LLDB from displaying union field contents.
validated versus https://godbolt.org/z/qrM37rY6n
…gross35 c-variadic: fix for sparc64 tracking issue: rust-lang#44930 Turns out it's a big-endian target that right-adjusts values in the stack slots. Apparently these tests do get run occasionally, though i don't think test failures are usually turned into issues on this repo. I guess we could add an assembly test here too, though really you just have to run these tests. I've tried this locally with qemu, and it passes all c-variadic tests. cc @thejpster @glaubitz r? tgross35
…cros, r=ehuss tests/ui/macros: add annotations for reference rules
Add docs about SDKs and C compilation on armv7a-vex-v5 This PR expands the documentation for armv7a-vex-v5 to fix broken links, add more information about SDKs, and create a new section about C compilation. I'm not super familiar with how Rust chooses what linker configuration to use / how the `cc` crate chooses what compiler to use - is this configuration to make C compilation work usually something that would be included in the compiler itself? Either way, it requires a lot of extra work which is kind of unfortunate (downloading a separate compiler, adding it to path, setting up a bunch of environment variables, and making a cargo config for linking).
Fix `get_child_at_index` return type hints returning `None` is valid when the index is invalid, so the type hints should reflect that
Fix array template arg lookup behavior Minor logic error. `get_template_args` expects a matching `<`/`>` pair. We were passing in an array type with the beginning `<` removed, so the template arg behavior would fail.
…l, r=ShoyuVanilla Const initialize `LOCK_LATCH` thread local A simple refactor to avoid runtime thread-local initialization.
Fix classify_union to return Union for regular unions Commit 623c7d7 accidentally changed the return value from REGULAR_UNION to RegularEnum when converting string literals to enum values. Commit b17670d then renamed RegularUnion to Union, but the buggy return statement remained unchanged. This caused unions to be misclassified as enums, preventing LLDB from displaying union field contents.
|
@bors r+ rollup=never p=5 |
|
The previous rollup (#155700) encountered a mysterious possibly-flaky failure in |
|
Oh, feel free to just reapprove that one and close this one. The failure is bizarre; #155518 was a similar failure. |
I think the old rollup is stale for other reasons, so this PR should be fine. |
This comment has been minimized.
This comment has been minimized.
|
📌 Perf builds for each rolled up PR:
previous master: 9836b06b55 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 9836b06 (parent) -> d493b7c (this PR) Test differencesShow 26 test diffsStage 1
Stage 2
Additionally, 2 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 d493b7c5ac637ed7edf626128b9f14796a2dad20 --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 (d493b7c): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis perf run didn't have relevant results for this metric. Max RSS (memory usage)Results (primary -1.9%, secondary -2.3%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary 2.0%, secondary -0.9%)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: 493.683s -> 491.485s (-0.45%) |
Successful merges:
get_child_at_indexreturn type hints #155685 (Fixget_child_at_indexreturn type hints)LOCK_LATCHthread local #155689 (Const initializeLOCK_LATCHthread local)r? @ghost
Create a similar rollup