Made benchmarking optional and removed some inlines.

dragostis · Sep 13, 2024 · 9a9a0d1 · 9a9a0d1
1 parent 7df2dce
commit 9a9a0d1
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 227 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -3,13 +3,14 @@ name = "spice"
 version = "0.1.0"
 edition = "2021"
 
-[dev-dependencies]
-divan = "0.1.14"
+[features]
+bench = ["dep:divan", "dep:rayon"]
+
+[dependencies]
+divan = { version = "0.1.14", optional = true }
+rayon = { version = "1.10.0", optional = true }
 
 [[bench]]
 name = "overhead"
 harness = false
-
-[dependencies]
-no_deadlocks = "1.3.2"
-rayon = "1.10.0"
+required-features = ["bench"]
diff --git a/benches/overhead.rs b/benches/overhead.rs
@@ -56,7 +56,6 @@ fn no_overhead(bencher: Bencher, nodes: (usize, usize)) {
 
 #[divan::bench(args = nodes())]
 fn overhead(bencher: Bencher, nodes: (usize, usize)) {
-    #[inline]
     fn sum(node: &Node, scope: &mut Scope<'_>) -> u64 {
         let (left, right) = scope.join(
             |s| node.left.as_deref().map(|n| sum(n, s)).unwrap_or_default(),
@@ -77,7 +76,6 @@ fn overhead(bencher: Bencher, nodes: (usize, usize)) {
 
 #[divan::bench(args = nodes())]
 fn rayon_overhead(bencher: Bencher, nodes: (usize, usize)) {
-    #[inline]
     fn sum(node: &Node) -> u64 {
         let (left, right) = rayon::join(
             || node.left.as_deref().map(sum).unwrap_or_default(),