Merge db96c9e6fb into 8eaa01b505

can't have coverage on SSE3/AVX2 instructions if CPU doesn't support it.

X10D.Tests/src/Numerics/NumberTests.cs

@@ -0,0 +1,32 @@
+#if NET7_0_OR_GREATER
+using NUnit.Framework;
+using X10D.Math;
+
+namespace X10D.Tests.Numerics;
+
+[TestFixture]
+internal class NumberTests
+{
+    [Test]
+    public void Sign_ShouldReturn1_GivenPositiveNumber()
+    {
+        Assert.That(NumberExtensions.Sign(2), Is.Positive);
+        Assert.That(NumberExtensions.Sign(2), Is.EqualTo(1));
+    }
+
+    [Test]
+    public void Sign_Should0_GivenZero()
+    {
+        Assert.That(NumberExtensions.Sign(0), Is.Not.Positive);
+        Assert.That(NumberExtensions.Sign(0), Is.Not.Negative);
+        Assert.That(NumberExtensions.Sign(0), Is.EqualTo(0));
+    }
+
+    [Test]
+    public void Sign_ShouldReturnNegative1_GivenNegativeNumber()
+    {
+        Assert.That(NumberExtensions.Sign(-2), Is.Negative);
+        Assert.That(NumberExtensions.Sign(-2), Is.EqualTo(-1));
+    }
+}
+#endif

X10D/src/Collections/BinaryIntegerExtensions.cs

@@ -1,4 +1,5 @@
 #if NET7_0_OR_GREATER
+using System.Diagnostics.CodeAnalysis;
 using System.Diagnostics.Contracts;
 using System.Numerics;
 using System.Runtime.CompilerServices;
@@ -48,6 +49,28 @@ public static class BinaryIntegerExtensions
             }
         }
 
+        UnpackInternal(value, destination);
+    }
+
+    [MethodImpl(CompilerResources.MaxOptimization)]
+    private static void UnpackInternal_Fallback<TInteger>(this TInteger value, Span<bool> destination)
+        where TInteger : unmanaged, IBinaryInteger<TInteger>
+    {
+        unsafe
+        {
+            int bitCount = sizeof(TInteger) * 8;
+            for (var index = 0; index < bitCount; index++)
+            {
+                destination[index] = (value & (TInteger.One << index)) != TInteger.Zero;
+            }
+        }
+    }
+
+    [ExcludeFromCodeCoverage]
+    [MethodImpl(CompilerResources.MaxOptimization)]
+    private static void UnpackInternal<TInteger>(TInteger value, Span<bool> destination)
+        where TInteger : unmanaged, IBinaryInteger<TInteger>
+    {
         switch (value)
         {
             case byte valueByte when Sse3.IsSupported:
@@ -71,19 +94,5 @@ public static class BinaryIntegerExtensions
                 break;
         }
     }
-
-    [MethodImpl(CompilerResources.MaxOptimization)]
-    internal static void UnpackInternal_Fallback<TInteger>(this TInteger value, Span<bool> destination)
-        where TInteger : unmanaged, IBinaryInteger<TInteger>
-    {
-        unsafe
-        {
-            int bitCount = sizeof(TInteger) * 8;
-            for (var index = 0; index < bitCount; index++)
-            {
-                destination[index] = (value & (TInteger.One << index)) != TInteger.Zero;
-            }
-        }
-    }
 }
 #endif