Unnamed repository; edit this file 'description' to name the repository.
Auto merge of #12704 - jonas-schievink:smol-paths, r=jonas-schievink
internal: Use `SmallVec` to slightly shrink `ModPath` size Saves like a megabyte on r-a itself.
bors 2022-07-07
parent 00194ad · parent d2fd137 · commit c46570e
-rw-r--r--Cargo.lock1
-rw-r--r--crates/hir-expand/Cargo.toml1
-rw-r--r--crates/hir-expand/src/mod_path.rs7
3 files changed, 6 insertions, 3 deletions
diff --git a/Cargo.lock b/Cargo.lock
index 69e27d2e81..fceaa0797c 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -532,6 +532,7 @@ dependencies = [
"mbe",
"profile",
"rustc-hash",
+ "smallvec",
"stdx",
"syntax",
"tracing",
diff --git a/crates/hir-expand/Cargo.toml b/crates/hir-expand/Cargo.toml
index 453afa6e2f..1fcc8287ed 100644
--- a/crates/hir-expand/Cargo.toml
+++ b/crates/hir-expand/Cargo.toml
@@ -19,6 +19,7 @@ itertools = "0.10.3"
hashbrown = { version = "0.12.1", features = [
"inline-more",
], default-features = false }
+smallvec = { version = "1.9.0", features = ["const_new"] }
stdx = { path = "../stdx", version = "0.0.0" }
base-db = { path = "../base-db", version = "0.0.0" }
diff --git a/crates/hir-expand/src/mod_path.rs b/crates/hir-expand/src/mod_path.rs
index af59733b9f..05e8c585c9 100644
--- a/crates/hir-expand/src/mod_path.rs
+++ b/crates/hir-expand/src/mod_path.rs
@@ -12,12 +12,13 @@ use crate::{
};
use base_db::CrateId;
use either::Either;
+use smallvec::SmallVec;
use syntax::{ast, AstNode};
#[derive(Debug, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
pub struct ModPath {
pub kind: PathKind,
- segments: Vec<Name>,
+ segments: SmallVec<[Name; 1]>,
}
#[derive(Debug, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
@@ -41,13 +42,13 @@ impl ModPath {
}
pub fn from_segments(kind: PathKind, segments: impl IntoIterator<Item = Name>) -> ModPath {
- let segments = segments.into_iter().collect::<Vec<_>>();
+ let segments = segments.into_iter().collect();
ModPath { kind, segments }
}
/// Creates a `ModPath` from a `PathKind`, with no extra path segments.
pub const fn from_kind(kind: PathKind) -> ModPath {
- ModPath { kind, segments: Vec::new() }
+ ModPath { kind, segments: SmallVec::new_const() }
}
pub fn segments(&self) -> &[Name] {