From f8d4cdc170bead42db3ffa647318ecf2bd6430e7 Mon Sep 17 00:00:00 2001 From: Shotaro Yamada Date: Mon, 14 Oct 2019 12:56:18 +0900 Subject: Avoid cloning `Arc<[T]>` into a vec if possible --- crates/ra_hir/src/ty/infer/unify.rs | 23 +++++++++++++++-------- crates/ra_hir/src/ty/lower.rs | 7 +++---- 2 files changed, 18 insertions(+), 12 deletions(-) (limited to 'crates/ra_hir/src/ty') diff --git a/crates/ra_hir/src/ty/infer/unify.rs b/crates/ra_hir/src/ty/infer/unify.rs index d161aa6b3..5e86ed260 100644 --- a/crates/ra_hir/src/ty/infer/unify.rs +++ b/crates/ra_hir/src/ty/infer/unify.rs @@ -6,6 +6,7 @@ use crate::ty::{ Canonical, InEnvironment, InferTy, ProjectionPredicate, ProjectionTy, Substs, TraitRef, Ty, TypeWalk, }; +use crate::util::make_mut_arc_slice; impl<'a, D: HirDatabase> InferenceContext<'a, D> { pub(super) fn canonicalizer<'b>(&'b mut self) -> Canonicalizer<'a, 'b, D> @@ -74,10 +75,13 @@ where }) } - fn do_canonicalize_trait_ref(&mut self, trait_ref: TraitRef) -> TraitRef { - let substs = - trait_ref.substs.iter().map(|ty| self.do_canonicalize_ty(ty.clone())).collect(); - TraitRef { trait_: trait_ref.trait_, substs: Substs(substs) } + fn do_canonicalize_trait_ref(&mut self, mut trait_ref: TraitRef) -> TraitRef { + make_mut_arc_slice(&mut trait_ref.substs.0, |tys| { + for ty in tys { + *ty = self.do_canonicalize_ty(ty.clone()); + } + }); + trait_ref } fn into_canonicalized(self, result: T) -> Canonicalized { @@ -87,10 +91,13 @@ where } } - fn do_canonicalize_projection_ty(&mut self, projection_ty: ProjectionTy) -> ProjectionTy { - let params = - projection_ty.parameters.iter().map(|ty| self.do_canonicalize_ty(ty.clone())).collect(); - ProjectionTy { associated_ty: projection_ty.associated_ty, parameters: Substs(params) } + fn do_canonicalize_projection_ty(&mut self, mut projection_ty: ProjectionTy) -> ProjectionTy { + make_mut_arc_slice(&mut projection_ty.parameters.0, |params| { + for ty in params { + *ty = self.do_canonicalize_ty(ty.clone()); + } + }); + projection_ty } fn do_canonicalize_projection_predicate( diff --git a/crates/ra_hir/src/ty/lower.rs b/crates/ra_hir/src/ty/lower.rs index a604c02e2..003aa9bab 100644 --- a/crates/ra_hir/src/ty/lower.rs +++ b/crates/ra_hir/src/ty/lower.rs @@ -392,10 +392,9 @@ impl TraitRef { ) -> Self { let mut substs = TraitRef::substs_from_path(db, resolver, segment, resolved); if let Some(self_ty) = explicit_self_ty { - // FIXME this could be nicer - let mut substs_vec = substs.0.to_vec(); - substs_vec[0] = self_ty; - substs.0 = substs_vec.into(); + crate::util::make_mut_arc_slice(&mut substs.0, |substs| { + substs[0] = self_ty; + }); } TraitRef { trait_: resolved, substs } } -- cgit v1.2.3 From 965ca0d271bdb467ec1f7fe309094aff8cac6aaa Mon Sep 17 00:00:00 2001 From: Shotaro Yamada Date: Mon, 14 Oct 2019 13:06:05 +0900 Subject: `.collect()` directly into `Arc<[T]>` --- crates/ra_hir/src/ty/lower.rs | 32 ++++++++++++++------------------ crates/ra_hir/src/ty/traits.rs | 2 +- crates/ra_hir/src/ty/traits/chalk.rs | 3 +-- 3 files changed, 16 insertions(+), 21 deletions(-) (limited to 'crates/ra_hir/src/ty') diff --git a/crates/ra_hir/src/ty/lower.rs b/crates/ra_hir/src/ty/lower.rs index 003aa9bab..2f9a2ee05 100644 --- a/crates/ra_hir/src/ty/lower.rs +++ b/crates/ra_hir/src/ty/lower.rs @@ -31,11 +31,11 @@ impl Ty { match type_ref { TypeRef::Never => Ty::simple(TypeCtor::Never), TypeRef::Tuple(inner) => { - let inner_tys = - inner.iter().map(|tr| Ty::from_hir(db, resolver, tr)).collect::>(); + let inner_tys: Arc<[Ty]> = + inner.iter().map(|tr| Ty::from_hir(db, resolver, tr)).collect(); Ty::apply( TypeCtor::Tuple { cardinality: inner_tys.len() as u16 }, - Substs(inner_tys.into()), + Substs(inner_tys), ) } TypeRef::Path(path) => Ty::from_hir_path(db, resolver, path), @@ -57,9 +57,7 @@ impl Ty { } TypeRef::Placeholder => Ty::Unknown, TypeRef::Fn(params) => { - let inner_tys = - params.iter().map(|tr| Ty::from_hir(db, resolver, tr)).collect::>(); - let sig = Substs(inner_tys.into()); + let sig = Substs(params.iter().map(|tr| Ty::from_hir(db, resolver, tr)).collect()); Ty::apply(TypeCtor::FnPtr { num_args: sig.len() as u16 - 1 }, sig) } TypeRef::DynTrait(bounds) => { @@ -69,8 +67,8 @@ impl Ty { .flat_map(|b| { GenericPredicate::from_type_bound(db, resolver, b, self_ty.clone()) }) - .collect::>(); - Ty::Dyn(predicates.into()) + .collect(); + Ty::Dyn(predicates) } TypeRef::ImplTrait(bounds) => { let self_ty = Ty::Bound(0); @@ -79,8 +77,8 @@ impl Ty { .flat_map(|b| { GenericPredicate::from_type_bound(db, resolver, b, self_ty.clone()) }) - .collect::>(); - Ty::Opaque(predicates.into()) + .collect(); + Ty::Opaque(predicates) } TypeRef::Error => Ty::Unknown, } @@ -557,13 +555,12 @@ pub(crate) fn generic_predicates_for_param_query( param_idx: u32, ) -> Arc<[GenericPredicate]> { let resolver = def.resolver(db); - let predicates = resolver + resolver .where_predicates_in_scope() // we have to filter out all other predicates *first*, before attempting to lower them .filter(|pred| Ty::from_hir_only_param(db, &resolver, &pred.type_ref) == Some(param_idx)) .flat_map(|pred| GenericPredicate::from_where_predicate(db, &resolver, pred)) - .collect::>(); - predicates.into() + .collect() } pub(crate) fn trait_env( @@ -584,11 +581,10 @@ pub(crate) fn generic_predicates_query( def: GenericDef, ) -> Arc<[GenericPredicate]> { let resolver = def.resolver(db); - let predicates = resolver + resolver .where_predicates_in_scope() .flat_map(|pred| GenericPredicate::from_where_predicate(db, &resolver, pred)) - .collect::>(); - predicates.into() + .collect() } /// Resolve the default type params from generics @@ -602,9 +598,9 @@ pub(crate) fn generic_defaults_query(db: &impl HirDatabase, def: GenericDef) -> .map(|p| { p.default.as_ref().map_or(Ty::Unknown, |path| Ty::from_hir_path(db, &resolver, path)) }) - .collect::>(); + .collect(); - Substs(defaults.into()) + Substs(defaults) } fn fn_sig_for_fn(db: &impl HirDatabase, def: Function) -> FnSig { diff --git a/crates/ra_hir/src/ty/traits.rs b/crates/ra_hir/src/ty/traits.rs index b0f67ae50..0cb5c3798 100644 --- a/crates/ra_hir/src/ty/traits.rs +++ b/crates/ra_hir/src/ty/traits.rs @@ -89,7 +89,7 @@ pub(crate) fn impls_for_trait_query( } let crate_impl_blocks = db.impls_in_crate(krate); impls.extend(crate_impl_blocks.lookup_impl_blocks_for_trait(trait_)); - impls.into_iter().collect::>().into() + impls.into_iter().collect() } /// A set of clauses that we assume to be true. E.g. if we are inside this function: diff --git a/crates/ra_hir/src/ty/traits/chalk.rs b/crates/ra_hir/src/ty/traits/chalk.rs index 9168de709..00aaf65d9 100644 --- a/crates/ra_hir/src/ty/traits/chalk.rs +++ b/crates/ra_hir/src/ty/traits/chalk.rs @@ -126,8 +126,7 @@ impl ToChalk for Substs { chalk_ir::Parameter(chalk_ir::ParameterKind::Ty(ty)) => from_chalk(db, ty), chalk_ir::Parameter(chalk_ir::ParameterKind::Lifetime(_)) => unimplemented!(), }) - .collect::>() - .into(); + .collect(); Substs(tys) } } -- cgit v1.2.3 From b462eb96b867cd38c60fb3d94ffd7688cec70f21 Mon Sep 17 00:00:00 2001 From: Shotaro Yamada Date: Mon, 14 Oct 2019 17:21:38 +0900 Subject: import make_mut_arc_slice --- crates/ra_hir/src/ty/lower.rs | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'crates/ra_hir/src/ty') diff --git a/crates/ra_hir/src/ty/lower.rs b/crates/ra_hir/src/ty/lower.rs index 2f9a2ee05..639518f27 100644 --- a/crates/ra_hir/src/ty/lower.rs +++ b/crates/ra_hir/src/ty/lower.rs @@ -22,6 +22,7 @@ use crate::{ resolve::{Resolver, TypeNs}, ty::Adt, type_ref::{TypeBound, TypeRef}, + util::make_mut_arc_slice, BuiltinType, Const, Enum, EnumVariant, Function, ModuleDef, Path, Static, Struct, StructField, Trait, TypeAlias, Union, }; @@ -390,7 +391,7 @@ impl TraitRef { ) -> Self { let mut substs = TraitRef::substs_from_path(db, resolver, segment, resolved); if let Some(self_ty) = explicit_self_ty { - crate::util::make_mut_arc_slice(&mut substs.0, |substs| { + make_mut_arc_slice(&mut substs.0, |substs| { substs[0] = self_ty; }); } -- cgit v1.2.3 From 3a55b5bf01ddc581a3f00fa56db725db93a131c6 Mon Sep 17 00:00:00 2001 From: Shotaro Yamada Date: Mon, 14 Oct 2019 19:50:12 +0900 Subject: make_mut_slice --- crates/ra_hir/src/ty/infer/unify.rs | 18 +++++++----------- crates/ra_hir/src/ty/lower.rs | 6 ++---- 2 files changed, 9 insertions(+), 15 deletions(-) (limited to 'crates/ra_hir/src/ty') diff --git a/crates/ra_hir/src/ty/infer/unify.rs b/crates/ra_hir/src/ty/infer/unify.rs index 5e86ed260..014c7981f 100644 --- a/crates/ra_hir/src/ty/infer/unify.rs +++ b/crates/ra_hir/src/ty/infer/unify.rs @@ -6,7 +6,7 @@ use crate::ty::{ Canonical, InEnvironment, InferTy, ProjectionPredicate, ProjectionTy, Substs, TraitRef, Ty, TypeWalk, }; -use crate::util::make_mut_arc_slice; +use crate::util::make_mut_slice; impl<'a, D: HirDatabase> InferenceContext<'a, D> { pub(super) fn canonicalizer<'b>(&'b mut self) -> Canonicalizer<'a, 'b, D> @@ -76,11 +76,9 @@ where } fn do_canonicalize_trait_ref(&mut self, mut trait_ref: TraitRef) -> TraitRef { - make_mut_arc_slice(&mut trait_ref.substs.0, |tys| { - for ty in tys { - *ty = self.do_canonicalize_ty(ty.clone()); - } - }); + for ty in make_mut_slice(&mut trait_ref.substs.0) { + *ty = self.do_canonicalize_ty(ty.clone()); + } trait_ref } @@ -92,11 +90,9 @@ where } fn do_canonicalize_projection_ty(&mut self, mut projection_ty: ProjectionTy) -> ProjectionTy { - make_mut_arc_slice(&mut projection_ty.parameters.0, |params| { - for ty in params { - *ty = self.do_canonicalize_ty(ty.clone()); - } - }); + for ty in make_mut_slice(&mut projection_ty.parameters.0) { + *ty = self.do_canonicalize_ty(ty.clone()); + } projection_ty } diff --git a/crates/ra_hir/src/ty/lower.rs b/crates/ra_hir/src/ty/lower.rs index 639518f27..366556134 100644 --- a/crates/ra_hir/src/ty/lower.rs +++ b/crates/ra_hir/src/ty/lower.rs @@ -22,7 +22,7 @@ use crate::{ resolve::{Resolver, TypeNs}, ty::Adt, type_ref::{TypeBound, TypeRef}, - util::make_mut_arc_slice, + util::make_mut_slice, BuiltinType, Const, Enum, EnumVariant, Function, ModuleDef, Path, Static, Struct, StructField, Trait, TypeAlias, Union, }; @@ -391,9 +391,7 @@ impl TraitRef { ) -> Self { let mut substs = TraitRef::substs_from_path(db, resolver, segment, resolved); if let Some(self_ty) = explicit_self_ty { - make_mut_arc_slice(&mut substs.0, |substs| { - substs[0] = self_ty; - }); + make_mut_slice(&mut substs.0)[0] = self_ty; } TraitRef { trait_: resolved, substs } } -- cgit v1.2.3