Do not throw for Marshal.SizeOf(typeof(delegate*<void>))

[MichalStrehovsky]

Noticed as I was looking at this code.

[dotnet-policy-service]

Tagging subscribers to this area: @dotnet/interop-contrib
See info in area-owners.md if you want to be subscribed.

[Copilot]

Pull Request Overview

This pull request addresses the issue of throwing exceptions when calling Marshal.SizeOf on function pointers by adding an appropriate test and updating the AOT helper method.

• Added a test in SizeOfTests.cs to validate delegate* functionality.
• Updated the NativeAot implementation to correctly recognize function pointers in the type-check condition.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/libraries/System.Runtime.InteropServices/tests/System.Runtime.InteropServices.UnitTests/System/Runtime/InteropServices/Marshal/SizeOfTests.cs	Added test for function pointer size computation
src/coreclr/nativeaot/System.Private.CoreLib/src/System/Runtime/InteropServices/Marshal.NativeAot.cs	Updated condition to handle function pointers in Marshal.SizeOf

Comments suppressed due to low confidence (1)

src/libraries/System.Runtime.InteropServices/tests/System.Runtime.InteropServices.UnitTests/System/Runtime/InteropServices/Marshal/SizeOfTests.cs:37

• [nitpick] Consider expanding test coverage to include additional function pointer variations, ensuring that the behavior remains consistent across different use cases.

Assert.Equal(IntPtr.Size, Marshal.SizeOf(typeof(delegate*<void>)));

[Copilot]

Consider removing the outdated comment fragment 'or IsFunctionPointer' since the condition now explicitly includes it, which would improve clarity.

[jkotas]

/ba-g unrelated failures (wasm)