Skip to content
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

Indicate change in RSS from start to end of pass in time-passes output #81771

Merged
merged 1 commit into from
Feb 5, 2021

Conversation

tgnottingham
Copy link
Contributor

Previously, this was omitted because it could be misleading, but the
functionality seems too useful not to include.

r? @oli-obk

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 5, 2021
@tgnottingham
Copy link
Contributor Author

Example output:

time:   0.000; rss:  145MB ->  145MB (   +0)    unsafety_checking
time:   0.000; rss:  145MB ->  145MB (   +0)    orphan_checking
time:   0.021; rss:  111MB ->  145MB (  +34)    coherence_checking
time:   0.001; rss:  145MB ->  148MB (   +3)    wf_checking
time:   0.001; rss:  148MB ->  148MB (   +0)    item_types_checking
time:   0.041; rss:  148MB ->  160MB (  +13)    item_bodies_checking
time:   0.065; rss:  111MB ->  160MB (  +49)    type_check_crate
time:   0.000; rss:  160MB ->  161MB (   +1)    match_checking
time:   0.000; rss:  161MB ->  161MB (   +0)    liveness_and_intrinsic_checking

@rustbot label T-compiler

@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Feb 5, 2021
@cjgillot
Copy link
Contributor

cjgillot commented Feb 5, 2021

Nit : the "MB" unit is missing inside the parenthesis.
Are there still cases of missing passes in the output ?

@tgnottingham
Copy link
Contributor Author

tgnottingham commented Feb 5, 2021

Are there still cases of missing passes in the output ?

I'm not aware of that issue. I only included a subset of the time-passes output in my previous comment for brevity's sake, if that's what you mean.

Nit : the "MB" unit is missing inside the parenthesis.

Fixed.

Previously, this was omitted because it could be misleading, but the
functionality seems too useful not to include.
@tgnottingham tgnottingham force-pushed the time-passes-rss-delta branch from bb86605 to 4253919 Compare February 5, 2021 09:12
@oli-obk
Copy link
Contributor

oli-obk commented Feb 5, 2021

@bors r+

@bors
Copy link
Contributor

bors commented Feb 5, 2021

📌 Commit 4253919 has been approved by oli-obk

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Feb 5, 2021
@oli-obk
Copy link
Contributor

oli-obk commented Feb 5, 2021

@bors rollup

m-ou-se added a commit to m-ou-se/rust that referenced this pull request Feb 5, 2021
… r=oli-obk

Indicate change in RSS from start to end of pass in time-passes output

Previously, this was omitted because it could be misleading, but the
functionality seems too useful not to include.

r? `@oli-obk`
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 5, 2021
Rollup of 15 pull requests

Successful merges:

 - rust-lang#79554 (Generic associated types in trait paths)
 - rust-lang#80726 (relax adt unsizing requirements)
 - rust-lang#81307 (Handle `Span`s for byte and raw strings and add more detail )
 - rust-lang#81318 (rustdoc-json: Fix has_body)
 - rust-lang#81456 (Make remote-test-server easier to use with new targets)
 - rust-lang#81497 (rustdoc: Move `display_fn` struct inside `display_fn`)
 - rust-lang#81500 (Remove struct_type from union output)
 - rust-lang#81542 (Expose correct symlink API on WASI)
 - rust-lang#81676 (Add more information to the error code for 'crate not found')
 - rust-lang#81682 (Add additional bitset benchmarks)
 - rust-lang#81730 (Make `Allocator` object-safe)
 - rust-lang#81763 (Cleanup rustdoc pass descriptions a bit)
 - rust-lang#81767 (Update LayoutError/LayoutErr stability attributes)
 - rust-lang#81771 (Indicate change in RSS from start to end of pass in time-passes output)
 - rust-lang#81781 (Fix `install-awscli.sh` error in CI)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 08d8fc1 into rust-lang:master Feb 5, 2021
@rustbot rustbot added this to the 1.51.0 milestone Feb 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants