Skip to content

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
Collaborator

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