simplify i16x16::move_mask (#151)

mcroomp · web-flow · commit 9f2f87813536 · 2024-04-29T12:54:00.000-06:00
diff --git a/src/i16x16_.rs b/src/i16x16_.rs
@@ -294,9 +294,10 @@ impl i16x16 {
   #[must_use]
   pub fn move_mask(self) -> i32 {
     pick! {
-      if #[cfg(target_feature="avx2")] {
-        (move_mask_i8_m256i(pack_i16_to_i8_m256i(self.avx2,shuffle_ai_i64_all_m256i::<0b01_00_11_10>(self.avx2))) & 0xffff) as i32
-      } else {
+      if #[cfg(target_feature="sse2")] {
+          let [a,b] = cast::<_,[m128i;2]>(self);
+          move_mask_i8_m128i( pack_i16_to_i8_m128i(a,b))
+        } else {
         self.a.move_mask() | (self.b.move_mask() << 8)
       }
     }

Original file line number	Diff line number	Diff line change
`@@ -294,9 +294,10 @@ impl i16x16 {`
`294`	`294`	`#[must_use]`
`295`	`295`	`pub fn move_mask(self) -> i32 {`
`296`	`296`	`pick! {`
`297`		`- if #[cfg(target_feature="avx2")] {`
`298`		`- (move_mask_i8_m256i(pack_i16_to_i8_m256i(self.avx2,shuffle_ai_i64_all_m256i::<0b01_00_11_10>(self.avx2))) & 0xffff) as i32`
`299`		`- } else {`
	`297`	`+ if #[cfg(target_feature="sse2")] {`
	`298`	`+ let [a,b] = cast::<_,[m128i;2]>(self);`
	`299`	`+ move_mask_i8_m128i( pack_i16_to_i8_m128i(a,b))`
	`300`	`+ } else {`
`300`	`301`	`self.a.move_mask() \| (self.b.move_mask() << 8)`
`301`	`302`	`}`
`302`	`303`	`}`