From 4ec87d5cd2024492faaa11bd121d39b943710357 Mon Sep 17 00:00:00 2001
From: Nicholas Nethercote <nnethercote@mozilla.com>
Date: Fri, 6 Dec 2019 13:37:35 +1100
Subject: [PATCH] Remove some `debug!` statements.

Because I am tired of looking at them.
---
 .../obligation_forest/mod.rs                  | 20 +------------------
 1 file changed, 1 insertion(+), 19 deletions(-)

diff --git a/src/librustc_data_structures/obligation_forest/mod.rs b/src/librustc_data_structures/obligation_forest/mod.rs
index 54b014a0f28..0f893252671 100644
--- a/src/librustc_data_structures/obligation_forest/mod.rs
+++ b/src/librustc_data_structures/obligation_forest/mod.rs
@@ -329,10 +329,7 @@ impl<O: ForestObligation> ObligationForest<O> {
 
         match self.active_cache.entry(obligation.as_predicate().clone()) {
             Entry::Occupied(o) => {
-                let index = *o.get();
-                debug!("register_obligation_at({:?}, {:?}) - duplicate of {:?}!",
-                       obligation, parent, index);
-                let node = &mut self.nodes[index];
+                let node = &mut self.nodes[*o.get()];
                 if let Some(parent_index) = parent {
                     // If the node is already in `active_cache`, it has already
                     // had its chance to be marked with a parent. So if it's
@@ -349,9 +346,6 @@ impl<O: ForestObligation> ObligationForest<O> {
                 }
             }
             Entry::Vacant(v) => {
-                debug!("register_obligation_at({:?}, {:?}) - ok, new index is {}",
-                       obligation, parent, self.nodes.len());
-
                 let obligation_tree_id = match parent {
                     Some(parent_index) => self.nodes[parent_index].obligation_tree_id,
                     None => self.obligation_tree_id_generator.next().unwrap(),
@@ -431,8 +425,6 @@ impl<O: ForestObligation> ObligationForest<O> {
                                   -> Outcome<O, P::Error>
         where P: ObligationProcessor<Obligation=O>
     {
-        debug!("process_obligations(len={})", self.nodes.len());
-
         self.gen += 1;
 
         let mut errors = vec![];
@@ -450,8 +442,6 @@ impl<O: ForestObligation> ObligationForest<O> {
         while index < self.nodes.len() {
             let node = &mut self.nodes[index];
 
-            debug!("process_obligations: node {} == {:?}", index, node);
-
             // `processor.process_obligation` can modify the predicate within
             // `node.obligation`, and that predicate is the key used for
             // `self.active_cache`. This means that `self.active_cache` can get
@@ -463,8 +453,6 @@ impl<O: ForestObligation> ObligationForest<O> {
             }
             let result = processor.process_obligation(&mut node.obligation);
 
-            debug!("process_obligations: node {} got result {:?}", index, result);
-
             match result {
                 ProcessResult::Unchanged => {
                     // No change in state.
@@ -511,8 +499,6 @@ impl<O: ForestObligation> ObligationForest<O> {
         self.process_cycles(processor);
         let completed = self.compress(do_completed);
 
-        debug!("process_obligations: complete");
-
         Outcome {
             completed,
             errors,
@@ -593,8 +579,6 @@ impl<O: ForestObligation> ObligationForest<O> {
     {
         let mut stack = vec![];
 
-        debug!("process_cycles()");
-
         for (index, node) in self.nodes.iter().enumerate() {
             // For some benchmarks this state test is extremely hot. It's a win
             // to handle the no-op cases immediately to avoid the cost of the
@@ -606,8 +590,6 @@ impl<O: ForestObligation> ObligationForest<O> {
             }
         }
 
-        debug!("process_cycles: complete");
-
         debug_assert!(stack.is_empty());
     }