Add function pointer type support to type loader #85287
Conversation
So that we can create new function pointer types at runtime within the context of `MakeGenericXXX`.
ghost
assigned 
|
Tagging subscribers to this area: @agocke, @MichalStrehovsky, @jkotas Issue DetailsSo that we can create new function pointer types at runtime within the context of Cc @dotnet/ilc-contrib
|
| @@ -2434,6 +2435,97 @@ public static void Run() | |||
| } | |||
| } | |||
|
|
|||
| class TestFunctionPointerLoading | |||
There was a problem hiding this comment.
We can use this test coverage on runtime flavors. Can this be regular tests that runs on all runtimes instead? (Either under src\tests or under src\libraries is fine.)
There was a problem hiding this comment.
I'm worried about people "cleaning up" the MakeGeneric part or the statics. This test is written in a very specific way because it's for Native AOT. Libraries devs don't know about these subtleties.
There was a problem hiding this comment.
(To be clear the concern is that this will be "cleaned up" in one of the 3000-line diffs when some analyzer enthusiast enables a new analyzer. I do look at things related to NativeAOT but I don't have the bandwidth to track the whole repo. The analyzer changes tend to cause subtle damage like this.)
So that we can create new function pointer types at runtime within the context of
MakeGenericXXX.Cc @dotnet/ilc-contrib