Rollup of 3 pull requests#154032
Conversation
Code that previously used `QueryStackFrame` now uses `TaggedQueryKey` directly. Code that previously used `Spanned<QueryStackFrame>` now uses `QueryStackFrame`, which includes a span.
These types have been unused since polymorphization was removed in <rust-lang#133883>.
Move query-stack-frame spans into `QueryStackFrame` - Follow-up to rust-lang#153907 (comment). --- Code that previously used `QueryStackFrame` now uses `TaggedQueryKey` directly. Code that previously used `Spanned<QueryStackFrame>` now uses `QueryStackFrame`, which includes a span. This ends up bringing QueryStackFrame closer to the expectations of the code that actually consumes query cycles, and also avoids the awkward and misleading `respan` calls and `node` field accesses that had been required by `Spanned`. r? nnethercote
Couple of driver interface improvements * Pass Session to `make_codegen_backend` callback. This simplifies some code in miri. * Move env/file_depinfo from ParseSess to Session. There is no reason it has to be in ParseSess rather than Session. * Rename hash_untracked_state to track_state to indicate that it isn't just used for hashing state, but also for adding env vars and files to be tracked through the dep info file.
Remove unused types `UnusedGenericParams` and `FiniteBitSet` - These types have been unused since polymorphization was removed in <rust-lang#133883>.
|
Rollup of everything. @bors r+ rollup=never p=5 |
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 91775db (parent) -> a3903b1 (this PR) Test differencesShow 8 test diffs8 doctest diffs were found. These are ignored, as they are noisy. Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard a3903b1e6f699ebe3662da79bd4a63bd699a3fea --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 (a3903b1): comparison URL. Overall result: ✅ 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.5%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (secondary -2.6%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (primary -0.1%, secondary -0.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 491.471s -> 481.409s (-2.05%) |
Successful merges:
QueryStackFrame#153998 (Move query-stack-frame spans intoQueryStackFrame)UnusedGenericParamsandFiniteBitSet#154026 (Remove unused typesUnusedGenericParamsandFiniteBitSet)r? @ghost
Create a similar rollup