Edit rustc_middle::dep_graph module documentation#80325
Edit rustc_middle::dep_graph module documentation#80325pierwill wants to merge 2 commits intorust-lang:masterfrom pierwill:rustcmiddle-depgraph-docs
Conversation
|
r? @estebank (rust-highfive has picked a reviewer for you, use r? to override) |
|
r? @cjgillot |
|
You may also want to edit the docs for |
There was a problem hiding this comment.
The define_dep_nodes macro is defined in dep_node.rs.
It defines the DepKind enum, and DepConstructor.
DepNode is defined in rustc_query_system::dep_graph::dep_node.
There was a problem hiding this comment.
Drive-by: DepConstructor should really just be a mod, since all it does is provide a namespace for functions constructing DepNodes.
There was a problem hiding this comment.
I am not sure the API is that restricted any more. There is an escape hatch DepNode::new_no_params which is used in non-incremental runs.
Maybe that module should link to |
|
☔ The latest upstream changes (presumably #78452) made this pull request unmergeable. Please resolve the merge conflicts. |
…erister Get rid of `DepConstructor` This removes fully 235 unused functions. Follow-up to rust-lang#80325 (comment). r? `@michaelwoerister` cc `@cjgillot`
…erister Get rid of `DepConstructor` This removes fully 235 unused functions. Follow-up to rust-lang#80325 (comment). r? ``@michaelwoerister`` cc ``@cjgillot``
This fixes the module-level docs not rendering. (As of this writing, the module description is missing from https://doc.rust-lang.org/nightly/nightly-rustc/rustc_middle/dep_graph/index.html).
|
The job Click to see the possible cause of the failure (guessed by this bot) |
|
Not at all sure I did this rebase correctly... 🤔 |
|
☔ The latest upstream changes (presumably #81093) made this pull request unmergeable. Please resolve the merge conflicts. |
The first commit here moves the content to
mod.rs, the second contains edits.The section I've called "DepNode creation and 'inference'" appears to have some outdated material. (
DepConstructoris a struct, not an enum.)