From 9fa08b22574dbb0cfb041d72b0de49758e899cfd Mon Sep 17 00:00:00 2001
From: Anton Firszov <antonfir@gmail.com>
Date: Fri, 26 Oct 2018 15:18:40 +0200
Subject: [PATCH] Vector4ConversionThreshold

---
 .../PixelFormats/PixelOperations{TPixel}.cs   | 20 +++++++++++++++++--
 1 file changed, 18 insertions(+), 2 deletions(-)

diff --git a/src/ImageSharp/PixelFormats/PixelOperations{TPixel}.cs b/src/ImageSharp/PixelFormats/PixelOperations{TPixel}.cs
index c3d8e23b9..668b2d031 100644
--- a/src/ImageSharp/PixelFormats/PixelOperations{TPixel}.cs
+++ b/src/ImageSharp/PixelFormats/PixelOperations{TPixel}.cs
@@ -17,6 +17,12 @@ namespace SixLabors.ImageSharp.PixelFormats
     public partial class PixelOperations<TPixel>
         where TPixel : struct, IPixel<TPixel>
     {
+        /// <summary>
+        /// It's not worth to bother the transitive pixel conversion method below this limit.
+        /// The value depends on the actual gain brought by the SIMD characteristics of the executing CPU and JIT.
+        /// </summary>
+        private static readonly int Vector4ConversionThreshold = CalculateVector4ConversionThreshold();
+
         /// <summary>
         /// Gets the global <see cref="PixelOperations{TPixel}"/> instance for the pixel type <typeparamref name="TPixel"/>
         /// </summary>
@@ -183,7 +189,7 @@ namespace SixLabors.ImageSharp.PixelFormats
             int count = sourcePixels.Length;
 
             // Not worth for small buffers:
-            if (count < 128)
+            if (count < Vector4ConversionThreshold)
             {
                 ToVector4DefaultImpl(sourcePixels, destVectors);
                 return;
@@ -218,7 +224,7 @@ namespace SixLabors.ImageSharp.PixelFormats
             int count = sourceVectors.Length;
 
             // Not worth for small buffers:
-            if (count < 128)
+            if (count < Vector4ConversionThreshold)
             {
                 FromVector4DefaultImpl(sourceVectors, destPixels);
                 return;
@@ -263,5 +269,15 @@ namespace SixLabors.ImageSharp.PixelFormats
                 dp = sp.ToVector4();
             }
         }
+
+        private static int CalculateVector4ConversionThreshold()
+        {
+            if (!Vector.IsHardwareAccelerated)
+            {
+                return int.MaxValue;
+            }
+
+            return SimdUtils.ExtendedIntrinsics.IsAvailable && SimdUtils.IsAvx2CompatibleArchitecture ? 256 : 128;
+        }
     }
 }
\ No newline at end of file