Replace Marshal.AllocHGlobal with NativeMemory.Alloc in cross-platform code

src/coreclr/tools/Common/JitInterface/CorInfoImpl.cs

@@ -3225,14 +3225,14 @@ private void reportMetadata(byte* key, void* value, nuint length)
         private void* allocateArray(UIntPtr cBytes)
 #pragma warning restore CA1822 // Mark members as static
         {
-            return (void*)Marshal.AllocHGlobal((IntPtr)(void*)cBytes);
+            return NativeMemory.Alloc((nuint)cBytes);
         }
 
 #pragma warning disable CA1822 // Mark members as static
         private void freeArray(void* array)
 #pragma warning restore CA1822 // Mark members as static
         {
-            Marshal.FreeHGlobal((IntPtr)array);
+            NativeMemory.Free(array);
         }
 
 #pragma warning disable CA1822 // Mark members as static

src/libraries/System.Runtime.InteropServices.JavaScript/src/System/Runtime/InteropServices/JavaScript/JSFunctionBinding.cs

@@ -417,7 +417,7 @@ internal static unsafe void DispatchJSImportAsyncPost(JSFunctionBinding signatur
             exc.slot.ReceiverShouldFree = true;
 
             var bytes = sizeof(JSMarshalerArgument) * arguments.Length;
-            void* cpy = (void*)Marshal.AllocHGlobal(bytes);
+            void* cpy = NativeMemory.Alloc((nuint)bytes);
             arguments.CopyTo(new Span<JSMarshalerArgument>(cpy, arguments.Length));
             var sig = (nint)signature.Header;
 
@@ -480,7 +480,7 @@ internal static unsafe void ResolveOrRejectPromise(JSProxyContext targetContext,
 
                 // this copy is freed in SystemInteropJS_ResolveOrRejectPromise
                 var bytes = sizeof(JSMarshalerArgument) * arguments.Length;
-                void* cpy = (void*)Marshal.AllocHGlobal(bytes);
+                void* cpy = NativeMemory.Alloc((nuint)bytes);
                 arguments.CopyTo(new Span<JSMarshalerArgument>(cpy, arguments.Length));
 
                 // async

src/libraries/System.Runtime.InteropServices.JavaScript/src/System/Runtime/InteropServices/JavaScript/JSHostImplementation.Types.cs

@@ -26,7 +26,7 @@ public PromiseHolder(JSProxyContext targetContext)
                 GCHandle = (IntPtr)InteropServices.GCHandle.Alloc(this, GCHandleType.Normal);
                 ProxyContext = targetContext;
 #if FEATURE_WASM_MANAGED_THREADS
-                State = (PromiseHolderState*)Marshal.AllocHGlobal(sizeof(PromiseHolderState));
+                State = (PromiseHolderState*)NativeMemory.Alloc((nuint)sizeof(PromiseHolderState));
                 Interlocked.Exchange(ref (*State).IsResolving, 0);
 #endif
             }
@@ -36,7 +36,7 @@ public PromiseHolder(JSProxyContext targetContext, nint gcvHandle)
                 GCHandle = gcvHandle;
                 ProxyContext = targetContext;
 #if FEATURE_WASM_MANAGED_THREADS
-                State = (PromiseHolderState*)Marshal.AllocHGlobal(sizeof(PromiseHolderState));
+                State = (PromiseHolderState*)NativeMemory.Alloc((nuint)sizeof(PromiseHolderState));
                 Interlocked.Exchange(ref (*State).IsResolving, 0);
 #endif
             }

src/libraries/System.Runtime.InteropServices.JavaScript/src/System/Runtime/InteropServices/JavaScript/JSHostImplementation.cs

@@ -131,7 +131,7 @@ public static unsafe JSFunctionBinding GetMethodSignature(ReadOnlySpan<JSMarshal
             }
 
             // this is never unallocated
-            IntPtr buffer = Marshal.AllocHGlobal(size);
+            IntPtr buffer = (IntPtr)NativeMemory.Alloc((nuint)size);
 
             var signature = new JSFunctionBinding
             {
@@ -185,7 +185,7 @@ public static unsafe JSFunctionBinding GetMethodSignature(ReadOnlySpan<JSMarshal
 
         public static unsafe void FreeMethodSignatureBuffer(JSFunctionBinding signature)
         {
-            Marshal.FreeHGlobal((nint)signature.Header);
+            NativeMemory.Free(signature.Header);
             signature.Header = null;
             signature.Sigs = null;
         }

src/libraries/System.Runtime.InteropServices.JavaScript/src/System/Runtime/InteropServices/JavaScript/JSProxyContext.cs

@@ -398,7 +398,7 @@ public void ReleasePromiseHolder(nint holderGCHandle)
 #if FEATURE_WASM_MANAGED_THREADS
                 unsafe
                 {
-                    Marshal.FreeHGlobal((IntPtr)holder.State);
+                    NativeMemory.Free(holder.State);
                     holder.State = null;
                 }
 #endif
@@ -442,7 +442,7 @@ public unsafe void ReleaseJSOwnedObjectByGCHandle(nint gcHandle)
                     holderCallback = holder.Callback;
                     holder.IsDisposed = true;
 #if FEATURE_WASM_MANAGED_THREADS
-                    Marshal.FreeHGlobal((IntPtr)holder.State);
+                    NativeMemory.Free(holder.State);
                     holder.State = null;
 #endif
                 }

src/libraries/System.Runtime.InteropServices.JavaScript/src/System/Runtime/InteropServices/JavaScript/Marshaling/JSMarshalerArgument.Byte.cs

@@ -83,7 +83,7 @@ public void ToJS(byte? value)
         /// It's used by JSImport code generator and should not be used by developers in source code.
         /// </summary>
         /// <param name="value">The value to be marshaled.</param>
-        public void ToManaged(out byte[]? value)
+        public unsafe void ToManaged(out byte[]? value)
         {
             if (slot.Type == MarshalerType.None)
             {
@@ -92,15 +92,15 @@ public void ToManaged(out byte[]? value)
             }
             value = new byte[slot.Length];
             Marshal.Copy(slot.IntPtrValue, value, 0, slot.Length);
-            Marshal.FreeHGlobal(slot.IntPtrValue);
+            NativeMemory.Free((void*)slot.IntPtrValue);
         }
 
         /// <summary>
         /// Implementation of the argument marshaling.
         /// It's used by JSImport code generator and should not be used by developers in source code.
         /// </summary>
         /// <param name="value">The value to be marshaled.</param>
-        public void ToJS(byte[]? value)
+        public unsafe void ToJS(byte[]? value)
         {
             if (value == null)
             {
@@ -109,7 +109,7 @@ public void ToJS(byte[]? value)
             }
             slot.Length = value.Length;
             slot.Type = MarshalerType.Array;
-            slot.IntPtrValue = Marshal.AllocHGlobal(value.Length * sizeof(byte));
+            slot.IntPtrValue = (IntPtr)NativeMemory.Alloc((nuint)(value.Length * sizeof(byte)));
             slot.ElementType = MarshalerType.Byte;
             Marshal.Copy(value, 0, slot.IntPtrValue, slot.Length);
         }

src/libraries/System.Runtime.InteropServices.JavaScript/src/System/Runtime/InteropServices/JavaScript/Marshaling/JSMarshalerArgument.Double.cs

@@ -86,7 +86,7 @@ public void ToJS(double? value)
 #if !DEBUG
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
 #endif
-        public void ToManaged(out double[]? value)
+        public unsafe void ToManaged(out double[]? value)
         {
             if (slot.Type == MarshalerType.None)
             {
@@ -95,7 +95,7 @@ public void ToManaged(out double[]? value)
             }
             value = new double[slot.Length];
             Marshal.Copy(slot.IntPtrValue, value, 0, slot.Length);
-            Marshal.FreeHGlobal(slot.IntPtrValue);
+            NativeMemory.Free((void*)slot.IntPtrValue);
         }
 
         /// <summary>
@@ -106,15 +106,15 @@ public void ToManaged(out double[]? value)
 #if !DEBUG
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
 #endif
-        public void ToJS(double[] value)
+        public unsafe void ToJS(double[] value)
         {
             if (value == null)
             {
                 slot.Type = MarshalerType.None;
                 return;
             }
             slot.Type = MarshalerType.Array;
-            slot.IntPtrValue = Marshal.AllocHGlobal(value.Length * sizeof(double));
+            slot.IntPtrValue = (IntPtr)NativeMemory.Alloc((nuint)(value.Length * sizeof(double)));
             slot.Length = value.Length;
             slot.ElementType = MarshalerType.Double;
             Marshal.Copy(value, 0, slot.IntPtrValue, slot.Length);

src/libraries/System.Runtime.InteropServices.JavaScript/src/System/Runtime/InteropServices/JavaScript/Marshaling/JSMarshalerArgument.Int32.cs

@@ -83,7 +83,7 @@ public void ToJS(int? value)
         /// It's used by JSImport code generator and should not be used by developers in source code.
         /// </summary>
         /// <param name="value">The value to be marshaled.</param>
-        public void ToManaged(out int[]? value)
+        public unsafe void ToManaged(out int[]? value)
         {
             if (slot.Type == MarshalerType.None)
             {
@@ -92,23 +92,23 @@ public void ToManaged(out int[]? value)
             }
             value = new int[slot.Length];
             Marshal.Copy(slot.IntPtrValue, value, 0, slot.Length);
-            Marshal.FreeHGlobal(slot.IntPtrValue);
+            NativeMemory.Free((void*)slot.IntPtrValue);
         }
 
         /// <summary>
         /// Implementation of the argument marshaling.
         /// It's used by JSImport code generator and should not be used by developers in source code.
         /// </summary>
         /// <param name="value">The value to be marshaled.</param>
-        public void ToJS(int[]? value)
+        public unsafe void ToJS(int[]? value)
         {
             if (value == null)
             {
                 slot.Type = MarshalerType.None;
                 return;
             }
             slot.Type = MarshalerType.Array;
-            slot.IntPtrValue = Marshal.AllocHGlobal(value.Length * sizeof(int));
+            slot.IntPtrValue = (IntPtr)NativeMemory.Alloc((nuint)(value.Length * sizeof(int)));
             slot.Length = value.Length;
             slot.ElementType = MarshalerType.Int32;
             Marshal.Copy(value, 0, slot.IntPtrValue, slot.Length);

src/libraries/System.Runtime.InteropServices.JavaScript/src/System/Runtime/InteropServices/JavaScript/Marshaling/JSMarshalerArgument.JSObject.cs

@@ -88,7 +88,7 @@ public unsafe void ToManaged(out JSObject?[]? value)
                 arg.ToManaged(out val);
                 value[i] = val;
             }
-            Marshal.FreeHGlobal(slot.IntPtrValue);
+            NativeMemory.Free((void*)slot.IntPtrValue);
         }
 
         /// <summary>
@@ -110,7 +110,7 @@ public unsafe void ToJS(JSObject?[] value)
             int bytes = value.Length * sizeof(JSMarshalerArgument);
             slot.Type = MarshalerType.Array;
             slot.ElementType = MarshalerType.JSObject;
-            JSMarshalerArgument* payload = (JSMarshalerArgument*)Marshal.AllocHGlobal(bytes);
+            JSMarshalerArgument* payload = (JSMarshalerArgument*)NativeMemory.Alloc((nuint)bytes);
             Unsafe.InitBlock(payload, 0, (uint)bytes);
             for (int i = 0; i < slot.Length; i++)
             {

src/libraries/System.Runtime.InteropServices.JavaScript/src/System/Runtime/InteropServices/JavaScript/Marshaling/JSMarshalerArgument.Object.cs

@@ -354,7 +354,7 @@ public unsafe void ToManaged(out object?[]? value)
 #if !ENABLE_JS_INTEROP_BY_VALUE
             Interop.Runtime.DeregisterGCRoot(slot.IntPtrValue);
 #endif
-            Marshal.FreeHGlobal(slot.IntPtrValue);
+            NativeMemory.Free((void*)slot.IntPtrValue);
         }
 
         /// <summary>
@@ -375,7 +375,7 @@ public unsafe void ToJS(object?[] value)
             slot.Length = value.Length;
             int bytes = value.Length * sizeof(JSMarshalerArgument);
             slot.Type = MarshalerType.Array;
-            JSMarshalerArgument* payload = (JSMarshalerArgument*)Marshal.AllocHGlobal(bytes);
+            JSMarshalerArgument* payload = (JSMarshalerArgument*)NativeMemory.Alloc((nuint)bytes);
             Unsafe.InitBlock(payload, 0, (uint)bytes);
 #if !ENABLE_JS_INTEROP_BY_VALUE
             Interop.Runtime.RegisterGCRoot(payload, bytes, IntPtr.Zero);

src/libraries/System.Runtime.InteropServices.JavaScript/src/System/Runtime/InteropServices/JavaScript/Marshaling/JSMarshalerArgument.String.cs

@@ -24,7 +24,7 @@ public unsafe void ToManaged(out string? value)
             }
 #if ENABLE_JS_INTEROP_BY_VALUE
             value = Marshal.PtrToStringUni(slot.IntPtrValue, slot.Length);
-            Marshal.FreeHGlobal(slot.IntPtrValue);
+            NativeMemory.Free((void*)slot.IntPtrValue);
 #else
             fixed (void* argAsRoot = &slot.IntPtrValue)
             {
@@ -96,7 +96,7 @@ public unsafe void ToManaged(out string?[]? value)
 #if !ENABLE_JS_INTEROP_BY_VALUE
             Interop.Runtime.DeregisterGCRoot(slot.IntPtrValue);
 #endif
-            Marshal.FreeHGlobal(slot.IntPtrValue);
+            NativeMemory.Free((void*)slot.IntPtrValue);
         }
 
         /// <summary>
@@ -117,7 +117,7 @@ public unsafe void ToJS(string?[] value)
             slot.Length = value.Length;
             int bytes = value.Length * sizeof(JSMarshalerArgument);
             slot.Type = MarshalerType.Array;
-            JSMarshalerArgument* payload = (JSMarshalerArgument*)Marshal.AllocHGlobal(bytes);
+            JSMarshalerArgument* payload = (JSMarshalerArgument*)NativeMemory.Alloc((nuint)bytes);
             Unsafe.InitBlock(payload, 0, (uint)bytes);
 #if !ENABLE_JS_INTEROP_BY_VALUE
             Interop.Runtime.RegisterGCRoot(payload, bytes, IntPtr.Zero);