Replace OptimizedInboxTextEncoder vectorization with SearchValues (#114494)

Author: MihaZupan

src/libraries/System.Text.Encodings.Web/src/System.Text.Encodings.Web.csproj

@@ -53,11 +53,6 @@ System.Text.Encodings.Web.JavaScriptEncoder</PackageDescription>
     <Compile Include="System\ThrowHelper.cs" />
   </ItemGroup>
 
-  <ItemGroup Condition="'$(TargetFrameworkIdentifier)' == '.NETCoreApp'">
-    <Compile Include="System\Text\Encodings\Web\OptimizedInboxTextEncoder.Ssse3.cs" />
-    <Compile Include="System\Text\Encodings\Web\OptimizedInboxTextEncoder.AdvSimd64.cs" />
-  </ItemGroup>
-
   <ItemGroup Condition="'$(TargetFrameworkIdentifier)' != '.NETCoreApp'">
     <Compile Include="$(CoreLibSharedDir)System\Text\Rune.cs" Link="System\Text\Rune.cs" />
     <Compile Include="$(CoreLibSharedDir)System\Text\Unicode\Utf16Utility.cs" Link="System\Text\Unicode\Utf16Utility.cs" />

src/libraries/System.Text.Encodings.Web/src/System/Text/Encodings/Web/OptimizedInboxTextEncoder.AdvSimd64.cs

@@ -3,71 +3,11 @@
 
 using System.Diagnostics;
 using System.Runtime.CompilerServices;
-using System.Runtime.InteropServices;
-
-#if NET
-using System.Runtime.Intrinsics;
-#endif
 
 namespace System.Text.Encodings.Web
 {
     internal sealed partial class OptimizedInboxTextEncoder
     {
-        /// <summary>
-        /// A bitmap which represents allowed ASCII code points.
-        /// </summary>
-        [StructLayout(LayoutKind.Explicit)]
-        private unsafe partial struct AllowedAsciiCodePoints
-        {
-            [FieldOffset(0)] // ensure same offset with AsVector field
-            private fixed byte AsBytes[16];
-
-#if NET
-#if !TARGET_BROWSER
-            [FieldOffset(0)] // ensure same offset with AsBytes field
-            internal Vector128<byte> AsVector;
-#else
-            // This member shouldn't be accessed from browser-based code paths.
-            // All call sites should be trimmed away, which will also trim this member
-            // and the type hierarchy it links to.
-#pragma warning disable CA1822
-            internal Vector128<byte> AsVector => throw new PlatformNotSupportedException();
-#pragma warning restore CA1822
-#endif
-#endif
-
-            [MethodImpl(MethodImplOptions.AggressiveInlining)]
-            internal readonly bool IsAllowedAsciiCodePoint(uint codePoint)
-            {
-                if (codePoint > 0x7F)
-                {
-                    return false; // non-ASCII
-                }
-
-                uint mask = AsBytes[codePoint & 0xF];
-                if ((mask & (0x1u << (int)(codePoint >> 4))) == 0)
-                {
-                    return false; // ASCII but disallowed
-                }
-
-                return true;
-            }
-
-            internal void PopulateAllowedCodePoints(in AllowedBmpCodePointsBitmap allowedBmpCodePoints)
-            {
-                this = default; // clear all existing data
-
-                // we only care about ASCII non-control chars; all control chars and non-ASCII chars are disallowed
-                for (int i = 0x20; i < 0x7F; i++)
-                {
-                    if (allowedBmpCodePoints.IsCharAllowed((char)i))
-                    {
-                        AsBytes[i & 0xF] |= (byte)(1 << (i >> 4));
-                    }
-                }
-            }
-        }
-
         /// <summary>
         /// A bitmap which represents the 64-bit pre-escaped form of the ASCII code points.
         /// A pre-escaped code point has the form [ WW 00 FF EE DD CC BB AA ],