Add check on sse2 and neon (#76)

* Add missing check on sse2 feature

* 3.3.2

* Cleanup

Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "gxhash"
 authors = ["Olivier Giniaux"]
-version = "3.3.1"
+version = "3.3.2"
 edition = "2021"
 description = "GxHash non-cryptographic algorithm"
 license = "MIT"

src/gxhash/platform/arm.rs

@@ -1,3 +1,6 @@
+#[cfg(not(any(all(target_feature = "aes", target_feature = "neon"), docsrs)))] // docs.rs bypasses the target_feature check
+compile_error!{"Gxhash requires aes and neon intrinsics. Make sure the processor supports it and build with RUSTFLAGS=\"-C target-cpu=native\" or RUSTFLAGS=\"-C target-feature=+aes,+neon\"."}
+
 #[cfg(target_arch = "arm")]
 use core::arch::arm::*;
 #[cfg(target_arch = "aarch64")]

src/gxhash/platform/mod.rs

@@ -1,10 +1,8 @@
 #[cfg(any(target_arch = "arm", target_arch = "aarch64"))]
-#[cfg(all(target_feature = "aes", target_feature = "neon"))]
 #[path = "arm.rs"]
 mod platform;
 
 #[cfg(any(target_arch = "x86", target_arch = "x86_64"))]
-#[cfg(any(all(target_feature = "aes", target_feature = "sse2"), docsrs))] // docsrs bypasses the target_feature check
 #[path = "x86.rs"]
 mod platform;
 

src/gxhash/platform/x86.rs

@@ -1,3 +1,9 @@
+#[cfg(not(any(all(target_feature = "aes", target_feature = "sse2"), docsrs)))] // docs.rs bypasses the target_feature check
+compile_error!{"Gxhash requires aes and sse2 intrinsics. Make sure the processor supports it and build with RUSTFLAGS=\"-C target-cpu=native\" or RUSTFLAGS=\"-C target-feature=+aes,+sse2\"."}
+
+#[cfg(all(feature = "hybrid", not(any(target_feature = "aes", target_feature = "vaes", target_feature = "avx2"))))]
+compile_error!{"Hybrid feature is only available on x86 processors with avx2 and vaes intrinsics."}
+
 #[cfg(target_arch = "x86")]
 use core::arch::x86::*;
 #[cfg(target_arch = "x86_64")]

src/lib.rs

@@ -2,12 +2,6 @@
 // Hybrid SIMD width usage currently requires unstable 'stdsimd'
 #![cfg_attr(feature = "hybrid", feature(stdarch_x86_avx512))]
 
-#[cfg(all(feature = "hybrid", not(any(target_arch = "x86_64", target_feature = "aes", target_feature = "vaes", target_feature = "avx2"))))]
-compile_error!{"Hybrid feature is only available on x86 processors with avx2 and vaes intrinsics."}
-
-#[cfg(not(target_feature = "aes"))]
-compile_error!{"Gxhash requires aes intrinsics. Make sure the processor supports it and build with RUSTFLAGS=\"-C target-cpu=native\" or RUSTFLAGS=\"-C target-feature=+aes\"."}
-
 #[rustfmt::skip]
 mod gxhash;
 pub use crate::gxhash::*;