]> git.lizzy.rs Git - rust.git/blob - src/librustc_data_structures/obligation_forest/mod.rs
Rollup merge of #67254 - RalfJung:vtable-ice, r=oli-obk
[rust.git] / src / librustc_data_structures / obligation_forest / mod.rs
1 //! The `ObligationForest` is a utility data structure used in trait
2 //! matching to track the set of outstanding obligations (those not yet
3 //! resolved to success or error). It also tracks the "backtrace" of each
4 //! pending obligation (why we are trying to figure this out in the first
5 //! place).
6 //!
7 //! ### External view
8 //!
9 //! `ObligationForest` supports two main public operations (there are a
10 //! few others not discussed here):
11 //!
12 //! 1. Add a new root obligations (`register_obligation`).
13 //! 2. Process the pending obligations (`process_obligations`).
14 //!
15 //! When a new obligation `N` is added, it becomes the root of an
16 //! obligation tree. This tree can also carry some per-tree state `T`,
17 //! which is given at the same time. This tree is a singleton to start, so
18 //! `N` is both the root and the only leaf. Each time the
19 //! `process_obligations` method is called, it will invoke its callback
20 //! with every pending obligation (so that will include `N`, the first
21 //! time). The callback also receives a (mutable) reference to the
22 //! per-tree state `T`. The callback should process the obligation `O`
23 //! that it is given and return a `ProcessResult`:
24 //!
25 //! - `Unchanged` -> ambiguous result. Obligation was neither a success
26 //!   nor a failure. It is assumed that further attempts to process the
27 //!   obligation will yield the same result unless something in the
28 //!   surrounding environment changes.
29 //! - `Changed(C)` - the obligation was *shallowly successful*. The
30 //!   vector `C` is a list of subobligations. The meaning of this is that
31 //!   `O` was successful on the assumption that all the obligations in `C`
32 //!   are also successful. Therefore, `O` is only considered a "true"
33 //!   success if `C` is empty. Otherwise, `O` is put into a suspended
34 //!   state and the obligations in `C` become the new pending
35 //!   obligations. They will be processed the next time you call
36 //!   `process_obligations`.
37 //! - `Error(E)` -> obligation failed with error `E`. We will collect this
38 //!   error and return it from `process_obligations`, along with the
39 //!   "backtrace" of obligations (that is, the list of obligations up to
40 //!   and including the root of the failed obligation). No further
41 //!   obligations from that same tree will be processed, since the tree is
42 //!   now considered to be in error.
43 //!
44 //! When the call to `process_obligations` completes, you get back an `Outcome`,
45 //! which includes three bits of information:
46 //!
47 //! - `completed`: a list of obligations where processing was fully
48 //!   completed without error (meaning that all transitive subobligations
49 //!   have also been completed). So, for example, if the callback from
50 //!   `process_obligations` returns `Changed(C)` for some obligation `O`,
51 //!   then `O` will be considered completed right away if `C` is the
52 //!   empty vector. Otherwise it will only be considered completed once
53 //!   all the obligations in `C` have been found completed.
54 //! - `errors`: a list of errors that occurred and associated backtraces
55 //!   at the time of error, which can be used to give context to the user.
56 //! - `stalled`: if true, then none of the existing obligations were
57 //!   *shallowly successful* (that is, no callback returned `Changed(_)`).
58 //!   This implies that all obligations were either errors or returned an
59 //!   ambiguous result, which means that any further calls to
60 //!   `process_obligations` would simply yield back further ambiguous
61 //!   results. This is used by the `FulfillmentContext` to decide when it
62 //!   has reached a steady state.
63 //!
64 //! ### Implementation details
65 //!
66 //! For the most part, comments specific to the implementation are in the
67 //! code. This file only contains a very high-level overview. Basically,
68 //! the forest is stored in a vector. Each element of the vector is a node
69 //! in some tree. Each node in the vector has the index of its dependents,
70 //! including the first dependent which is known as the parent. It also
71 //! has a current state, described by `NodeState`. After each processing
72 //! step, we compress the vector to remove completed and error nodes, which
73 //! aren't needed anymore.
74
75 use crate::fx::{FxHashMap, FxHashSet};
76
77 use std::cell::{Cell, RefCell};
78 use std::collections::hash_map::Entry;
79 use std::fmt::Debug;
80 use std::hash;
81 use std::marker::PhantomData;
82
83 mod graphviz;
84
85 #[cfg(test)]
86 mod tests;
87
88 pub trait ForestObligation : Clone + Debug {
89     type Predicate : Clone + hash::Hash + Eq + Debug;
90
91     fn as_predicate(&self) -> &Self::Predicate;
92 }
93
94 pub trait ObligationProcessor {
95     type Obligation : ForestObligation;
96     type Error : Debug;
97
98     fn process_obligation(&mut self,
99                           obligation: &mut Self::Obligation)
100                           -> ProcessResult<Self::Obligation, Self::Error>;
101
102     /// As we do the cycle check, we invoke this callback when we
103     /// encounter an actual cycle. `cycle` is an iterator that starts
104     /// at the start of the cycle in the stack and walks **toward the
105     /// top**.
106     ///
107     /// In other words, if we had O1 which required O2 which required
108     /// O3 which required O1, we would give an iterator yielding O1,
109     /// O2, O3 (O1 is not yielded twice).
110     fn process_backedge<'c, I>(&mut self,
111                                cycle: I,
112                                _marker: PhantomData<&'c Self::Obligation>)
113         where I: Clone + Iterator<Item=&'c Self::Obligation>;
114 }
115
116 /// The result type used by `process_obligation`.
117 #[derive(Debug)]
118 pub enum ProcessResult<O, E> {
119     Unchanged,
120     Changed(Vec<O>),
121     Error(E),
122 }
123
124 #[derive(Clone, Copy, PartialEq, Eq, Hash, Debug)]
125 struct ObligationTreeId(usize);
126
127 type ObligationTreeIdGenerator =
128     ::std::iter::Map<::std::ops::RangeFrom<usize>, fn(usize) -> ObligationTreeId>;
129
130 pub struct ObligationForest<O: ForestObligation> {
131     /// The list of obligations. In between calls to `process_obligations`,
132     /// this list only contains nodes in the `Pending` or `Success` state.
133     ///
134     /// `usize` indices are used here and throughout this module, rather than
135     /// `rustc_index::newtype_index!` indices, because this code is hot enough
136     /// that the `u32`-to-`usize` conversions that would be required are
137     /// significant, and space considerations are not important.
138     nodes: Vec<Node<O>>,
139
140     /// The process generation is 1 on the first call to `process_obligations`,
141     /// 2 on the second call, etc.
142     gen: u32,
143
144     /// A cache of predicates that have been successfully completed.
145     done_cache: FxHashSet<O::Predicate>,
146
147     /// A cache of the nodes in `nodes`, indexed by predicate. Unfortunately,
148     /// its contents are not guaranteed to match those of `nodes`. See the
149     /// comments in `process_obligation` for details.
150     active_cache: FxHashMap<O::Predicate, usize>,
151
152     /// A vector reused in compress(), to avoid allocating new vectors.
153     node_rewrites: RefCell<Vec<usize>>,
154
155     obligation_tree_id_generator: ObligationTreeIdGenerator,
156
157     /// Per tree error cache. This is used to deduplicate errors,
158     /// which is necessary to avoid trait resolution overflow in
159     /// some cases.
160     ///
161     /// See [this][details] for details.
162     ///
163     /// [details]: https://github.com/rust-lang/rust/pull/53255#issuecomment-421184780
164     error_cache: FxHashMap<ObligationTreeId, FxHashSet<O::Predicate>>,
165 }
166
167 #[derive(Debug)]
168 struct Node<O> {
169     obligation: O,
170     state: Cell<NodeState>,
171
172     /// Obligations that depend on this obligation for their completion. They
173     /// must all be in a non-pending state.
174     dependents: Vec<usize>,
175
176     /// If true, dependents[0] points to a "parent" node, which requires
177     /// special treatment upon error but is otherwise treated the same.
178     /// (It would be more idiomatic to store the parent node in a separate
179     /// `Option<usize>` field, but that slows down the common case of
180     /// iterating over the parent and other descendants together.)
181     has_parent: bool,
182
183     /// Identifier of the obligation tree to which this node belongs.
184     obligation_tree_id: ObligationTreeId,
185 }
186
187 impl<O> Node<O> {
188     fn new(
189         parent: Option<usize>,
190         obligation: O,
191         obligation_tree_id: ObligationTreeId
192     ) -> Node<O> {
193         Node {
194             obligation,
195             state: Cell::new(NodeState::Pending),
196             dependents:
197                 if let Some(parent_index) = parent {
198                     vec![parent_index]
199                 } else {
200                     vec![]
201                 },
202             has_parent: parent.is_some(),
203             obligation_tree_id,
204         }
205     }
206 }
207
208 /// The state of one node in some tree within the forest. This
209 /// represents the current state of processing for the obligation (of
210 /// type `O`) associated with this node.
211 ///
212 /// The non-`Error` state transitions are as follows.
213 /// ```
214 /// (Pre-creation)
215 ///  |
216 ///  |     register_obligation_at() (called by process_obligations() and
217 ///  v                               from outside the crate)
218 /// Pending
219 ///  |
220 ///  |     process_obligations()
221 ///  v
222 /// Success(not_waiting())
223 ///  |  |
224 ///  |  |  mark_still_waiting_nodes()
225 ///  |  v
226 ///  | Success(still_waiting())
227 ///  |  |
228 ///  |  |  compress()
229 ///  v  v
230 /// (Removed)
231 /// ```
232 /// The `Error` state can be introduced in several places, via `error_at()`.
233 ///
234 /// Outside of `ObligationForest` methods, nodes should be either `Pending` or
235 /// `Success`.
236 #[derive(Debug, Copy, Clone, PartialEq, Eq)]
237 enum NodeState {
238     /// This obligation has not yet been selected successfully. Cannot have
239     /// subobligations.
240     Pending,
241
242     /// This obligation was selected successfully, but it may be waiting on one
243     /// or more pending subobligations, as indicated by the `WaitingState`.
244     Success(WaitingState),
245
246     /// This obligation was resolved to an error. It will be removed by the
247     /// next compression step.
248     Error,
249 }
250
251 /// Indicates when a `Success` node was last (if ever) waiting on one or more
252 /// `Pending` nodes. The notion of "when" comes from `ObligationForest::gen`.
253 /// - 0: "Not waiting". This is a special value, set by `process_obligation`,
254 ///   and usable because generation counting starts at 1.
255 /// - 1..ObligationForest::gen: "Was waiting" in a previous generation, but
256 ///   waiting no longer. In other words, finished.
257 /// - ObligationForest::gen: "Still waiting" in this generation.
258 ///
259 /// Things to note about this encoding:
260 /// - Every time `ObligationForest::gen` is incremented, all the "still
261 ///   waiting" nodes automatically become "was waiting".
262 /// - `ObligationForest::is_still_waiting` is very cheap.
263 ///
264 #[derive(Clone, Copy, Debug, PartialEq, Eq, PartialOrd)]
265 struct WaitingState(u32);
266
267 #[derive(Debug)]
268 pub struct Outcome<O, E> {
269     /// Obligations that were completely evaluated, including all
270     /// (transitive) subobligations. Only computed if requested.
271     pub completed: Option<Vec<O>>,
272
273     /// Backtrace of obligations that were found to be in error.
274     pub errors: Vec<Error<O, E>>,
275
276     /// If true, then we saw no successful obligations, which means
277     /// there is no point in further iteration. This is based on the
278     /// assumption that when trait matching returns `Error` or
279     /// `Unchanged`, those results do not affect environmental
280     /// inference state. (Note that if we invoke `process_obligations`
281     /// with no pending obligations, stalled will be true.)
282     pub stalled: bool,
283 }
284
285 /// Should `process_obligations` compute the `Outcome::completed` field of its
286 /// result?
287 #[derive(PartialEq)]
288 pub enum DoCompleted {
289     No,
290     Yes,
291 }
292
293 #[derive(Debug, PartialEq, Eq)]
294 pub struct Error<O, E> {
295     pub error: E,
296     pub backtrace: Vec<O>,
297 }
298
299 impl<O: ForestObligation> ObligationForest<O> {
300     pub fn new() -> ObligationForest<O> {
301         ObligationForest {
302             nodes: vec![],
303             gen: 0,
304             done_cache: Default::default(),
305             active_cache: Default::default(),
306             node_rewrites: RefCell::new(vec![]),
307             obligation_tree_id_generator: (0..).map(ObligationTreeId),
308             error_cache: Default::default(),
309         }
310     }
311
312     /// Returns the total number of nodes in the forest that have not
313     /// yet been fully resolved.
314     pub fn len(&self) -> usize {
315         self.nodes.len()
316     }
317
318     /// Registers an obligation.
319     pub fn register_obligation(&mut self, obligation: O) {
320         // Ignore errors here - there is no guarantee of success.
321         let _ = self.register_obligation_at(obligation, None);
322     }
323
324     // Returns Err(()) if we already know this obligation failed.
325     fn register_obligation_at(&mut self, obligation: O, parent: Option<usize>) -> Result<(), ()> {
326         if self.done_cache.contains(obligation.as_predicate()) {
327             return Ok(());
328         }
329
330         match self.active_cache.entry(obligation.as_predicate().clone()) {
331             Entry::Occupied(o) => {
332                 let node = &mut self.nodes[*o.get()];
333                 if let Some(parent_index) = parent {
334                     // If the node is already in `active_cache`, it has already
335                     // had its chance to be marked with a parent. So if it's
336                     // not already present, just dump `parent` into the
337                     // dependents as a non-parent.
338                     if !node.dependents.contains(&parent_index) {
339                         node.dependents.push(parent_index);
340                     }
341                 }
342                 if let NodeState::Error = node.state.get() {
343                     Err(())
344                 } else {
345                     Ok(())
346                 }
347             }
348             Entry::Vacant(v) => {
349                 let obligation_tree_id = match parent {
350                     Some(parent_index) => self.nodes[parent_index].obligation_tree_id,
351                     None => self.obligation_tree_id_generator.next().unwrap(),
352                 };
353
354                 let already_failed =
355                     parent.is_some()
356                         && self.error_cache
357                             .get(&obligation_tree_id)
358                             .map(|errors| errors.contains(obligation.as_predicate()))
359                             .unwrap_or(false);
360
361                 if already_failed {
362                     Err(())
363                 } else {
364                     let new_index = self.nodes.len();
365                     v.insert(new_index);
366                     self.nodes.push(Node::new(parent, obligation, obligation_tree_id));
367                     Ok(())
368                 }
369             }
370         }
371     }
372
373     /// Converts all remaining obligations to the given error.
374     pub fn to_errors<E: Clone>(&mut self, error: E) -> Vec<Error<O, E>> {
375         let errors = self.nodes.iter().enumerate()
376             .filter(|(_index, node)| node.state.get() == NodeState::Pending)
377             .map(|(index, _node)| {
378                 Error {
379                     error: error.clone(),
380                     backtrace: self.error_at(index),
381                 }
382             })
383             .collect();
384
385         let successful_obligations = self.compress(DoCompleted::Yes);
386         assert!(successful_obligations.unwrap().is_empty());
387         errors
388     }
389
390     /// Returns the set of obligations that are in a pending state.
391     pub fn map_pending_obligations<P, F>(&self, f: F) -> Vec<P>
392         where F: Fn(&O) -> P
393     {
394         self.nodes.iter()
395             .filter(|node| node.state.get() == NodeState::Pending)
396             .map(|node| f(&node.obligation))
397             .collect()
398     }
399
400     fn insert_into_error_cache(&mut self, index: usize) {
401         let node = &self.nodes[index];
402         self.error_cache
403             .entry(node.obligation_tree_id)
404             .or_default()
405             .insert(node.obligation.as_predicate().clone());
406     }
407
408     fn not_waiting() -> WaitingState {
409         WaitingState(0)
410     }
411
412     fn still_waiting(&self) -> WaitingState {
413         WaitingState(self.gen)
414     }
415
416     fn is_still_waiting(&self, waiting: WaitingState) -> bool {
417         waiting.0 == self.gen
418     }
419
420     /// Performs a pass through the obligation list. This must
421     /// be called in a loop until `outcome.stalled` is false.
422     ///
423     /// This _cannot_ be unrolled (presently, at least).
424     pub fn process_obligations<P>(&mut self, processor: &mut P, do_completed: DoCompleted)
425                                   -> Outcome<O, P::Error>
426         where P: ObligationProcessor<Obligation=O>
427     {
428         self.gen += 1;
429
430         let mut errors = vec![];
431         let mut stalled = true;
432
433         // Note that the loop body can append new nodes, and those new nodes
434         // will then be processed by subsequent iterations of the loop.
435         //
436         // We can't use an iterator for the loop because `self.nodes` is
437         // appended to and the borrow checker would complain. We also can't use
438         // `for index in 0..self.nodes.len() { ... }` because the range would
439         // be computed with the initial length, and we would miss the appended
440         // nodes. Therefore we use a `while` loop.
441         let mut index = 0;
442         while index < self.nodes.len() {
443             let node = &mut self.nodes[index];
444
445             // `processor.process_obligation` can modify the predicate within
446             // `node.obligation`, and that predicate is the key used for
447             // `self.active_cache`. This means that `self.active_cache` can get
448             // out of sync with `nodes`. It's not very common, but it does
449             // happen, and code in `compress` has to allow for it.
450             if node.state.get() != NodeState::Pending {
451                 index += 1;
452                 continue;
453             }
454
455             match processor.process_obligation(&mut node.obligation) {
456                 ProcessResult::Unchanged => {
457                     // No change in state.
458                 }
459                 ProcessResult::Changed(children) => {
460                     // We are not (yet) stalled.
461                     stalled = false;
462                     node.state.set(NodeState::Success(Self::not_waiting()));
463
464                     for child in children {
465                         let st = self.register_obligation_at(
466                             child,
467                             Some(index)
468                         );
469                         if let Err(()) = st {
470                             // Error already reported - propagate it
471                             // to our node.
472                             self.error_at(index);
473                         }
474                     }
475                 }
476                 ProcessResult::Error(err) => {
477                     stalled = false;
478                     errors.push(Error {
479                         error: err,
480                         backtrace: self.error_at(index),
481                     });
482                 }
483             }
484             index += 1;
485         }
486
487         if stalled {
488             // There's no need to perform marking, cycle processing and compression when nothing
489             // changed.
490             return Outcome {
491                 completed: if do_completed == DoCompleted::Yes { Some(vec![]) } else { None },
492                 errors,
493                 stalled,
494             };
495         }
496
497         self.mark_still_waiting_nodes();
498         self.process_cycles(processor);
499         let completed = self.compress(do_completed);
500
501         Outcome {
502             completed,
503             errors,
504             stalled,
505         }
506     }
507
508     /// Returns a vector of obligations for `p` and all of its
509     /// ancestors, putting them into the error state in the process.
510     fn error_at(&self, mut index: usize) -> Vec<O> {
511         let mut error_stack: Vec<usize> = vec![];
512         let mut trace = vec![];
513
514         loop {
515             let node = &self.nodes[index];
516             node.state.set(NodeState::Error);
517             trace.push(node.obligation.clone());
518             if node.has_parent {
519                 // The first dependent is the parent, which is treated
520                 // specially.
521                 error_stack.extend(node.dependents.iter().skip(1));
522                 index = node.dependents[0];
523             } else {
524                 // No parent; treat all dependents non-specially.
525                 error_stack.extend(node.dependents.iter());
526                 break;
527             }
528         }
529
530         while let Some(index) = error_stack.pop() {
531             let node = &self.nodes[index];
532             if node.state.get() != NodeState::Error {
533                 node.state.set(NodeState::Error);
534                 error_stack.extend(node.dependents.iter());
535             }
536         }
537
538         trace
539     }
540
541     /// Mark all `Success` nodes that depend on a pending node as still
542     /// waiting. Upon completion, any `Success` nodes that aren't still waiting
543     /// can be removed by `compress`.
544     fn mark_still_waiting_nodes(&self) {
545         for node in &self.nodes {
546             if node.state.get() == NodeState::Pending {
547                 // This call site is hot.
548                 self.inlined_mark_dependents_as_still_waiting(node);
549             }
550         }
551     }
552
553     // This always-inlined function is for the hot call site.
554     #[inline(always)]
555     fn inlined_mark_dependents_as_still_waiting(&self, node: &Node<O>) {
556         for &index in node.dependents.iter() {
557             let node = &self.nodes[index];
558             if let NodeState::Success(waiting) = node.state.get() {
559                 if !self.is_still_waiting(waiting) {
560                     node.state.set(NodeState::Success(self.still_waiting()));
561                     // This call site is cold.
562                     self.uninlined_mark_dependents_as_still_waiting(node);
563                 }
564             }
565         }
566     }
567
568     // This never-inlined function is for the cold call site.
569     #[inline(never)]
570     fn uninlined_mark_dependents_as_still_waiting(&self, node: &Node<O>) {
571         self.inlined_mark_dependents_as_still_waiting(node)
572     }
573
574     /// Report cycles between all `Success` nodes that aren't still waiting.
575     /// This must be called after `mark_still_waiting_nodes`.
576     fn process_cycles<P>(&self, processor: &mut P)
577         where P: ObligationProcessor<Obligation=O>
578     {
579         let mut stack = vec![];
580
581         for (index, node) in self.nodes.iter().enumerate() {
582             // For some benchmarks this state test is extremely hot. It's a win
583             // to handle the no-op cases immediately to avoid the cost of the
584             // function call.
585             if let NodeState::Success(waiting) = node.state.get() {
586                 if !self.is_still_waiting(waiting) {
587                     self.find_cycles_from_node(&mut stack, processor, index, index);
588                 }
589             }
590         }
591
592         debug_assert!(stack.is_empty());
593     }
594
595     fn find_cycles_from_node<P>(&self, stack: &mut Vec<usize>, processor: &mut P, min_index: usize,
596                                 index: usize)
597         where P: ObligationProcessor<Obligation=O>
598     {
599         let node = &self.nodes[index];
600         if let NodeState::Success(waiting) = node.state.get() {
601             if !self.is_still_waiting(waiting) {
602                 match stack.iter().rposition(|&n| n == index) {
603                     None => {
604                         stack.push(index);
605                         for &dep_index in node.dependents.iter() {
606                             // The index check avoids re-considering a node.
607                             if dep_index >= min_index {
608                                 self.find_cycles_from_node(stack, processor, min_index, dep_index);
609                             }
610                         }
611                         stack.pop();
612                     }
613                     Some(rpos) => {
614                         // Cycle detected.
615                         processor.process_backedge(
616                             stack[rpos..].iter().map(GetObligation(&self.nodes)),
617                             PhantomData
618                         );
619                     }
620                 }
621             }
622         }
623     }
624
625     /// Compresses the vector, removing all popped nodes. This adjusts the
626     /// indices and hence invalidates any outstanding indices. `process_cycles`
627     /// must be run beforehand to remove any cycles on not-still-waiting
628     /// `Success` nodes.
629     #[inline(never)]
630     fn compress(&mut self, do_completed: DoCompleted) -> Option<Vec<O>> {
631         let orig_nodes_len = self.nodes.len();
632         let mut node_rewrites: Vec<_> = self.node_rewrites.replace(vec![]);
633         debug_assert!(node_rewrites.is_empty());
634         node_rewrites.extend(0..orig_nodes_len);
635         let mut dead_nodes = 0;
636         let mut removed_success_obligations: Vec<O> = vec![];
637
638         // Move removable nodes to the end, preserving the order of the
639         // remaining nodes.
640         //
641         // LOOP INVARIANT:
642         //     self.nodes[0..index - dead_nodes] are the first remaining nodes
643         //     self.nodes[index - dead_nodes..index] are all dead
644         //     self.nodes[index..] are unchanged
645         for index in 0..orig_nodes_len {
646             let node = &self.nodes[index];
647             match node.state.get() {
648                 NodeState::Pending => {
649                     if dead_nodes > 0 {
650                         self.nodes.swap(index, index - dead_nodes);
651                         node_rewrites[index] -= dead_nodes;
652                     }
653                 }
654                 NodeState::Success(waiting) if self.is_still_waiting(waiting) => {
655                     if dead_nodes > 0 {
656                         self.nodes.swap(index, index - dead_nodes);
657                         node_rewrites[index] -= dead_nodes;
658                     }
659                 }
660                 NodeState::Success(_) => {
661                     // This lookup can fail because the contents of
662                     // `self.active_cache` are not guaranteed to match those of
663                     // `self.nodes`. See the comment in `process_obligation`
664                     // for more details.
665                     if let Some((predicate, _)) =
666                         self.active_cache.remove_entry(node.obligation.as_predicate())
667                     {
668                         self.done_cache.insert(predicate);
669                     } else {
670                         self.done_cache.insert(node.obligation.as_predicate().clone());
671                     }
672                     if do_completed == DoCompleted::Yes {
673                         // Extract the success stories.
674                         removed_success_obligations.push(node.obligation.clone());
675                     }
676                     node_rewrites[index] = orig_nodes_len;
677                     dead_nodes += 1;
678                 }
679                 NodeState::Error => {
680                     // We *intentionally* remove the node from the cache at this point. Otherwise
681                     // tests must come up with a different type on every type error they
682                     // check against.
683                     self.active_cache.remove(node.obligation.as_predicate());
684                     self.insert_into_error_cache(index);
685                     node_rewrites[index] = orig_nodes_len;
686                     dead_nodes += 1;
687                 }
688             }
689         }
690
691         if dead_nodes > 0 {
692             // Remove the dead nodes and rewrite indices.
693             self.nodes.truncate(orig_nodes_len - dead_nodes);
694             self.apply_rewrites(&node_rewrites);
695         }
696
697         node_rewrites.truncate(0);
698         self.node_rewrites.replace(node_rewrites);
699
700         if do_completed == DoCompleted::Yes {
701             Some(removed_success_obligations)
702         } else {
703             None
704         }
705     }
706
707     fn apply_rewrites(&mut self, node_rewrites: &[usize]) {
708         let orig_nodes_len = node_rewrites.len();
709
710         for node in &mut self.nodes {
711             let mut i = 0;
712             while i < node.dependents.len() {
713                 let new_index = node_rewrites[node.dependents[i]];
714                 if new_index >= orig_nodes_len {
715                     node.dependents.swap_remove(i);
716                     if i == 0 && node.has_parent {
717                         // We just removed the parent.
718                         node.has_parent = false;
719                     }
720                 } else {
721                     node.dependents[i] = new_index;
722                     i += 1;
723                 }
724             }
725         }
726
727         // This updating of `self.active_cache` is necessary because the
728         // removal of nodes within `compress` can fail. See above.
729         self.active_cache.retain(|_predicate, index| {
730             let new_index = node_rewrites[*index];
731             if new_index >= orig_nodes_len {
732                 false
733             } else {
734                 *index = new_index;
735                 true
736             }
737         });
738     }
739 }
740
741 // I need a Clone closure.
742 #[derive(Clone)]
743 struct GetObligation<'a, O>(&'a [Node<O>]);
744
745 impl<'a, 'b, O> FnOnce<(&'b usize,)> for GetObligation<'a, O> {
746     type Output = &'a O;
747     extern "rust-call" fn call_once(self, args: (&'b usize,)) -> &'a O {
748         &self.0[*args.0].obligation
749     }
750 }
751
752 impl<'a, 'b, O> FnMut<(&'b usize,)> for GetObligation<'a, O> {
753     extern "rust-call" fn call_mut(&mut self, args: (&'b usize,)) -> &'a O {
754         &self.0[*args.0].obligation
755     }
756 }