From b09bc88e9f9b659f903be1ee23227a04e683b234 Mon Sep 17 00:00:00 2001 From: Havvy Date: Sat, 9 Jun 2018 15:09:07 -0700 Subject: [PATCH] Refactor: Rename ExistentialPredicate::cmp to ExistentialPredicate::stable_cmp See https://github.com/rust-lang/rust/pull/51276#discussion_r193549404 for rationale. --- src/librustc/ty/context.rs | 2 +- src/librustc/ty/sty.rs | 4 +++- src/librustc_typeck/astconv.rs | 2 +- 3 files changed, 5 insertions(+), 3 deletions(-) diff --git a/src/librustc/ty/context.rs b/src/librustc/ty/context.rs index e66ad24231094..44e1b51714d14 100644 --- a/src/librustc/ty/context.rs +++ b/src/librustc/ty/context.rs @@ -2514,7 +2514,7 @@ impl<'a, 'gcx, 'tcx> TyCtxt<'a, 'gcx, 'tcx> { pub fn intern_existential_predicates(self, eps: &[ExistentialPredicate<'tcx>]) -> &'tcx Slice> { assert!(!eps.is_empty()); - assert!(eps.windows(2).all(|w| w[0].cmp(self, &w[1]) != Ordering::Greater)); + assert!(eps.windows(2).all(|w| w[0].stable_cmp(self, &w[1]) != Ordering::Greater)); self._intern_existential_predicates(eps) } diff --git a/src/librustc/ty/sty.rs b/src/librustc/ty/sty.rs index f484fda2ae152..8afa96bf2dfe3 100644 --- a/src/librustc/ty/sty.rs +++ b/src/librustc/ty/sty.rs @@ -496,7 +496,9 @@ pub enum ExistentialPredicate<'tcx> { } impl<'a, 'gcx, 'tcx> ExistentialPredicate<'tcx> { - pub fn cmp(&self, tcx: TyCtxt<'a, 'gcx, 'tcx>, other: &Self) -> Ordering { + /// Compares via an ordering that will not change if modules are reordered or other changes are + /// made to the tree. In particular, this ordering is preserved across incremental compilations. + pub fn stable_cmp(&self, tcx: TyCtxt<'a, 'gcx, 'tcx>, other: &Self) -> Ordering { use self::ExistentialPredicate::*; match (*self, *other) { (Trait(_), Trait(_)) => Ordering::Equal, diff --git a/src/librustc_typeck/astconv.rs b/src/librustc_typeck/astconv.rs index 7ef510f41254f..479fe2a20b10d 100644 --- a/src/librustc_typeck/astconv.rs +++ b/src/librustc_typeck/astconv.rs @@ -714,7 +714,7 @@ impl<'o, 'gcx: 'tcx, 'tcx> AstConv<'gcx, 'tcx>+'o { .chain(existential_projections .map(|x| ty::ExistentialPredicate::Projection(*x.skip_binder()))) .collect::>(); - v.sort_by(|a, b| a.cmp(tcx, b)); + v.sort_by(|a, b| a.stable_cmp(tcx, b)); let existential_predicates = ty::Binder::bind(tcx.mk_existential_predicates(v.into_iter()));