Commit 37f9991

authored

chore: hide avx512 behind feature flag (#39)

* chore: hide avx512 behind feature flag * clippy

1 parent 1d36b0c commit 37f9991Copy full SHA for 37f9991

File tree

+12

-5

lines changed

+12

-5

lines changed

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@ members = ["cpu-features"]`
`5`	`5`	`name = "json-escape-simd"`
`6`	`6`	`version = "3.0.0"`
`7`	`7`	`edition = "2024"`
`8`		`-rust-version = "1.89.0"`
	`8`	`+rust-version = "1.85.0"`
`9`	`9`	`include = ["src/*/.rs"]`
`10`	`10`	`description = "Optimized SIMD routines for escaping JSON strings."`
`11`	`11`	`license = "MIT"`
`@@ -18,6 +18,7 @@ path = "examples/escape.rs"`
`18`	`18`
`19`	`19`	`[features]`
`20`	`20`	`codspeed = ["criterion2/codspeed"]`
	`21`	`+avx512 = []`
`21`	`22`	`asan = [] # for ASAN`
`22`	`23`
`23`	`24`	`[[bench]]`

Lines changed: 9 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -2,6 +2,8 @@`
`2`	`2`	`//!`
`3`	`3`	`//! Only takes the string escaping part to avoid the abstraction overhead.`
`4`	`4`
	`5`	`+#![allow(clippy::incompatible_msrv)]`
	`6`	`+`
`5`	`7`	`#[cfg(any(target_arch = "x86_64", target_arch = "x86"))]`
`6`	`8`	`use std::arch::is_x86_feature_detected;`
`7`	`9`
`@@ -297,9 +299,13 @@ fn format_string(value: &str, dst: &mut [u8]) -> usize {`
`297`	`299`
`298`	`300`	`#[cfg(any(target_arch = "x86", target_arch = "x86_64"))]`
`299`	`301`	`{`
`300`		`- if is_x86_feature_detected!("avx512f") {`
`301`		`- unsafe { simd::avx512::format_string(value, dst) }`
`302`		`- } else if is_x86_feature_detected!("avx2") {`
	`302`	`+ #[cfg(feature = "avx512")]`
	`303`	`+ {`
	`304`	`+ if is_x86_feature_detected!("avx512f") {`
	`305`	`+ return unsafe { simd::avx512::format_string(value, dst) };`
	`306`	`+ }`
	`307`	`+ }`
	`308`	`+ if is_x86_feature_detected!("avx2") {`
`303`	`309`	`unsafe { simd::avx2::format_string(value, dst) }`
`304`	`310`	`} else if is_x86_feature_detected!("sse2") {`
`305`	`311`	`unsafe { simd::sse2::format_string(value, dst) }`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,7 @@`
`2`	`2`
`3`	`3`	`#[cfg(any(target_arch = "x86_64", target_arch = "x86"))]`
`4`	`4`	`pub(crate) mod avx2;`
`5`		`-#[cfg(any(target_arch = "x86_64", target_arch = "x86"))]`
	`5`	`+#[cfg(all(any(target_arch = "x86_64", target_arch = "x86"), feature = "avx512"))]`
`6`	`6`	`pub(crate) mod avx512;`
`7`	`7`	`pub mod bits;`
`8`	`8`	`#[cfg(target_arch = "aarch64")]`

Comments

(0)