-
Notifications
You must be signed in to change notification settings - Fork 13.5k
Rollup of 10 pull requests #142795
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rollup of 10 pull requests #142795
Conversation
Signed-off-by: xizheyin <[email protected]>
…e-case use consistent title capitalization
Adjust some doc for Query System
content has moved to another chapter
use sentence case
Query cleaning
Use stage 1 for building docs
Also replaced '→' symbol with '->' for consistency across the table.
Remove hanging parenthesis from example signature.
Profiling with perf: specify the section of bootstrap settings.
Document Ambig vs Unambig Type/Consts
That was intended to be a list. Also, the order is not relevant.
initial instructions for gpu offload
…ingjubilee Convert `ilog(10)` to `ilog10()` Except in tests, convert `integer.ilog(10)` to `integer.ilog10()` for better speed and to provide better examples of code that efficiently counts decimal digits. I couldn't find any instances of `integer.ilog(2)`.
…rochenkov Some symbol and PathRoot cleanups I'm looking into unifying how we join and print paths. Here are some preliminary cleanups. r? ``@petrochenkov``
…thod, r=tgross35 remove equivalent new method on context Noticed the two had converged so much they literally became equivalent. So one could go hehe
@bors r+ rollup=never p=5 |
☀️ Test successful - checks-actions |
📌 Perf builds for each rolled up PR:
previous master: 9c4ff566ba 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 9c4ff56 (parent) -> 5526a2f (this PR) Test differencesShow 2 test diffs2 doctest diffs were found. These are ignored, as they are noisy. Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard 5526a2f47cd676ceeedc08cf71ae75ce2e9284ae --output-dir test-dashboard And 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 (5526a2f): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)Results (primary 3.3%, secondary -2.8%)A less reliable metric. May be of interest, but not used to determine the overall result above.
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: 692.606s -> 692.994s (0.06%) |
Successful merges:
#[align]
attribute #142715 (correct template for#[align]
attribute)ExternalCrate
methods #142720 (De-dup common code fromExternalCrate
methods)src/rustdoc-json-types
is changed #142758 (Make sure to rebuild rustdoc ifsrc/rustdoc-json-types
is changed)ilog(10)
toilog10()
#142764 (Convertilog(10)
toilog10()
)r? @ghost
@rustbot modify labels: rollup
Create a similar rollup