Fix <unknown> queries and add more timing info to render_html#81275
Fix <unknown> queries and add more timing info to render_html#81275bors merged 4 commits intorust-lang:masterfrom
Conversation
|
Another idea I thought of - rather than making each tool call this on its own, it might be easier to add it to the |
This comment has been minimized.
This comment has been minimized.
|
I think |
Hmm, for some reason this didn't work. |
This comment has been minimized.
This comment has been minimized.
It turns out this is |
Ugh - so the modified version of this idea I came up with, add it to DetailsFor now I reverted the change and just had rustdoc call it manually. I was playing with adding it to |
This comment has been minimized.
This comment has been minimized.
e697a60 to
1e7c4f3
Compare
|
The job Click to see the possible cause of the failure (guessed by this bot) |
This is no longer necessary now that rustdoc doesn't run everybody_loops.
I unreverted this and removed the call to |
This happened because `alloc_query_strings` was never called.
wesleywiser
left a comment
There was a problem hiding this comment.
One small piece of feedback. Feel free to r=me after!
- Show `create_renderer` and `renderer_after_crate` by default - Don't rewrite `extra_verbose_generic_activity`
This avoids each tool having to separately find and call `self_profile_alloc_strings`. - Don't compute the global context if it hasn't yet been computed This avoids giving extraneous errors about unresolved names if an error occurs during parsing.
|
@bors r=wesleywiser Thanks for the review! |
|
📌 Commit ca72f9e has been approved by |
…as-schievink Rollup of 15 pull requests Successful merges: - rust-lang#79841 (More clear documentation for NonNull<T>) - rust-lang#81072 (PlaceRef::ty: use method call syntax) - rust-lang#81130 (Edit rustc_middle::dep_graph module documentation) - rust-lang#81170 (Avoid hash_slice in VecDeque's Hash implementation) - rust-lang#81243 (mir: Improve size_of handling when arg is unsized) - rust-lang#81245 (Update cargo) - rust-lang#81249 (Lower closure prototype after its body.) - rust-lang#81252 (Add more self-profile info to rustc_resolve) - rust-lang#81275 (Fix <unknown> queries and add more timing info to render_html) - rust-lang#81281 (Inline methods of Path and OsString) - rust-lang#81283 (Note library tracking issue template in tracking issue template.) - rust-lang#81285 (Remove special casing of rustdoc in rustc_lint) - rust-lang#81288 (rustdoc: Fix visibility of trait and impl items) - rust-lang#81298 (replace RefCell with Cell in FnCtxt) - rust-lang#81301 (Fix small typo) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
Make `-Z time-passes` less noisy
- Add the module name to `pre_AST_expansion_passes` and don't make it a
verbose event (since it normally doesn't take very long, and it's
emitted many times)
- Don't make the following rustdoc events verbose; they're emitted many times.
+ build_extern_trait_impl
+ build_local_trait_impl
+ build_primitive_trait_impl
+ get_auto_trait_impls
+ get_blanket_trait_impls
- Remove the `get_auto_trait_and_blanket_synthetic_impls` rustdoc event; it's wholly
covered by get_{auto,blanket}_trait_impls and not very useful.
I found this while working on rust-lang#81275 but it's independent of those changes.
Make `-Z time-passes` less noisy
- Add the module name to `pre_AST_expansion_passes` and don't make it a
verbose event (since it normally doesn't take very long, and it's
emitted many times)
- Don't make the following rustdoc events verbose; they're emitted many times.
+ build_extern_trait_impl
+ build_local_trait_impl
+ build_primitive_trait_impl
+ get_auto_trait_impls
+ get_blanket_trait_impls
- Remove the `get_auto_trait_and_blanket_synthetic_impls` rustdoc event; it's wholly
covered by get_{auto,blanket}_trait_impls and not very useful.
I found this while working on rust-lang#81275 but it's independent of those changes.
Make `-Z time-passes` less noisy
- Add the module name to `pre_AST_expansion_passes` and don't make it a
verbose event (since it normally doesn't take very long, and it's
emitted many times)
- Don't make the following rustdoc events verbose; they're emitted many times.
+ build_extern_trait_impl
+ build_local_trait_impl
+ build_primitive_trait_impl
+ get_auto_trait_impls
+ get_blanket_trait_impls
- Remove the `get_auto_trait_and_blanket_synthetic_impls` rustdoc event; it's wholly
covered by get_{auto,blanket}_trait_impls and not very useful.
I found this while working on rust-lang#81275 but it's independent of those changes.
Closes #81251.
Fix
<unknown>queriesThis happened because
alloc_query_stringswas never called.Add more timing info to render_html
This still has some issues I'm not sure how to work out:
create_rendererandrenderer_after_kratearen't shown by default.I want something like
verbose_generic_activity_with_arg, but it doesn't exist.I'm also not sure how to show activities that aren't on by default - I
tried
-Z self-profile -Z self-profile-args=all, but it didn't show up.r? @wesleywiser