Skip to content

Commit

Permalink
Auto merge of #54486 - orium:obligation-forest-graphviz, r=nikomatsakis
Browse files Browse the repository at this point in the history
Added graphviz visualization for obligation forests.

This can be a big help when debugging the trait resolver.
  • Loading branch information
bors committed Oct 16, 2018
2 parents 8a7048b + 3fc275d commit bef62cc
Show file tree
Hide file tree
Showing 5 changed files with 106 additions and 0 deletions.
1 change: 1 addition & 0 deletions src/Cargo.lock
Original file line number Diff line number Diff line change
Expand Up @@ -2161,6 +2161,7 @@ version = "0.0.0"
dependencies = [
"cfg-if 0.1.5 (registry+https://github.com/rust-lang/crates.io-index)",
"ena 0.9.3 (registry+https://github.com/rust-lang/crates.io-index)",
"graphviz 0.0.0",
"log 0.4.4 (registry+https://github.com/rust-lang/crates.io-index)",
"parking_lot 0.6.4 (registry+https://github.com/rust-lang/crates.io-index)",
"parking_lot_core 0.2.14 (registry+https://github.com/rust-lang/crates.io-index)",
Expand Down
1 change: 1 addition & 0 deletions src/librustc_data_structures/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ ena = "0.9.3"
log = "0.4"
rustc_cratesio_shim = { path = "../librustc_cratesio_shim" }
serialize = { path = "../libserialize" }
graphviz = { path = "../libgraphviz" }
cfg-if = "0.1.2"
stable_deref_trait = "1.0.0"
parking_lot_core = "0.2.8"
Expand Down
1 change: 1 addition & 0 deletions src/librustc_data_structures/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@ extern crate rustc_rayon as rayon;
extern crate rustc_rayon_core as rayon_core;
extern crate rustc_hash;
extern crate serialize;
extern crate graphviz;
extern crate smallvec;

// See librustc_cratesio_shim/Cargo.toml for a comment explaining this.
Expand Down
101 changes: 101 additions & 0 deletions src/librustc_data_structures/obligation_forest/graphviz.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,101 @@
// Copyright 2018 The Rust Project Developers. See the COPYRIGHT
// file at the top-level directory of this distribution and at
// http://rust-lang.org/COPYRIGHT.
//
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
// option. This file may not be copied, modified, or distributed
// except according to those terms.

use graphviz as dot;
use obligation_forest::{ForestObligation, ObligationForest};
use std::env::var_os;
use std::fs::File;
use std::path::Path;
use std::sync::atomic::AtomicUsize;
use std::sync::atomic::Ordering;

impl<O: ForestObligation> ObligationForest<O> {
/// Create a graphviz representation of the obligation forest. Given a directory this will
/// create files with name of the format `<counter>_<description>.gv`. The counter is
/// global and is maintained internally.
///
/// Calling this will do nothing unless the environment variable
/// `DUMP_OBLIGATION_FOREST_GRAPHVIZ` is defined.
///
/// A few post-processing that you might want to do make the forest easier to visualize:
///
/// * `sed 's,std::[a-z]*::,,g'` — Deletes the `std::<package>::` prefix of paths.
/// * `sed 's,"Binder(TraitPredicate(<\(.*\)>)) (\([^)]*\))","\1 (\2)",'` — Transforms
/// `Binder(TraitPredicate(<predicate>))` into just `<predicate>`.
#[allow(dead_code)]
pub fn dump_graphviz<P: AsRef<Path>>(&self, dir: P, description: &str) {
static COUNTER: AtomicUsize = AtomicUsize::new(0);

if var_os("DUMP_OBLIGATION_FOREST_GRAPHVIZ").is_none() {
return;
}

let counter = COUNTER.fetch_add(1, Ordering::AcqRel);

let file_path = dir.as_ref().join(format!("{:010}_{}.gv", counter, description));

let mut gv_file = File::create(file_path).unwrap();

dot::render(&self, &mut gv_file).unwrap();
}
}

impl<'a, O: ForestObligation + 'a> dot::Labeller<'a> for &'a ObligationForest<O> {
type Node = usize;
type Edge = (usize, usize);

fn graph_id(&self) -> dot::Id {
dot::Id::new("trait_obligation_forest").unwrap()
}

fn node_id(&self, index: &Self::Node) -> dot::Id {
dot::Id::new(format!("obligation_{}", index)).unwrap()
}

fn node_label(&self, index: &Self::Node) -> dot::LabelText {
let node = &self.nodes[*index];
let label = format!("{:?} ({:?})", node.obligation.as_predicate(), node.state.get());

dot::LabelText::LabelStr(label.into())
}

fn edge_label(&self, (_index_source, _index_target): &Self::Edge) -> dot::LabelText {
dot::LabelText::LabelStr("".into())
}
}

impl<'a, O: ForestObligation + 'a> dot::GraphWalk<'a> for &'a ObligationForest<O> {
type Node = usize;
type Edge = (usize, usize);

fn nodes(&self) -> dot::Nodes<Self::Node> {
(0..self.nodes.len()).collect()
}

fn edges(&self) -> dot::Edges<Self::Edge> {
(0..self.nodes.len())
.flat_map(|i| {
let node = &self.nodes[i];

node.parent.iter().map(|p| p.get())
.chain(node.dependents.iter().map(|p| p.get()))
.map(move |p| (p, i))
})
.collect()
}

fn source(&self, (s, _): &Self::Edge) -> Self::Node {
*s
}

fn target(&self, (_, t): &Self::Edge) -> Self::Node {
*t
}
}
2 changes: 2 additions & 0 deletions src/librustc_data_structures/obligation_forest/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,8 @@ use std::marker::PhantomData;
mod node_index;
use self::node_index::NodeIndex;

mod graphviz;

#[cfg(test)]
mod test;

Expand Down

0 comments on commit bef62cc

Please sign in to comment.