Skip to content

[release/6.0] [monoapi] Add mono_method_get_unmanaged_callers_only_ftnptr - #66299

Merged
ericstj merged 1 commit into
dotnet:release/6.0from
lambdageek:backport-gh-66007-6.0
Mar 10, 2022
Merged

[release/6.0] [monoapi] Add mono_method_get_unmanaged_callers_only_ftnptr#66299
ericstj merged 1 commit into
dotnet:release/6.0from
lambdageek:backport-gh-66007-6.0

Conversation

@lambdageek

Copy link
Copy Markdown
Member

Backport of #66007

Customer Impact

As a performance improvement, exposes an additional Mono embedding API function that allows .NET for Android to create unmanaged function pointers to managed UnmanagedCallersOnly methods from native code, avoiding an extra roundtrip to managed.

Testing

Manual testing

Risk

Very Low. This is an additional unmanaged API entrypoint there is no relationship to existing customer application code.

Like `RuntimeMethodHandle.GetFunctionPointer`, but callable from native code
@ghostghost assigned lambdageekMar 7, 2022
@ghostghost added the area-VM-meta-mono label Mar 7, 2022
@lambdageeklambdageek added this to the 6.0.MAUI milestone Mar 7, 2022
@lambdageeklambdageek added the Servicing-consider Issue for next servicing release review label Mar 7, 2022
@lambdageeklambdageek added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Mar 9, 2022
@lambdageek

Copy link
Copy Markdown
MemberAuthor

Approved by tactics in email

@lambdageeklambdageek modified the milestones: 6.0.MAUI, 6.0.xMar 10, 2022
@lewing

Copy link
Copy Markdown
Member

wasm AOT failure is #66366

@ericstj
ericstj merged commit 8a48736 into dotnet:release/6.0Mar 10, 2022
@ericstjericstj modified the milestones: 6.0.x, 6.0.4Mar 10, 2022
@ghostghost locked as resolved and limited conversation to collaborators Apr 10, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-VM-meta-monoServicing-approvedApproved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@lambdageek@lewing@thaystg@ericstj