Skip to content

Implement missing SymUnmanaged* interfaces - #66650

Merged
agocke merged 5 commits into
dotnet:mainfrom
hez2010:nativeaot/sym-interface
Mar 17, 2022
Merged

Implement missing SymUnmanaged* interfaces#66650
agocke merged 5 commits into
dotnet:mainfrom
hez2010:nativeaot/sym-interface

Conversation

@hez2010

Copy link
Copy Markdown
Contributor

This PR implements missing RCW COMWrappers for SymUnmanaged* interfaces required by compiling BenchmarkDotNet with NativeAOT.

Fixes#66637

/cc: @agocke

@ghostghost added area-NativeAOT-coreclr community-contribution Indicates that the PR has been added by a community member labels Mar 15, 2022

@agockeagocke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@agocke

Copy link
Copy Markdown
Member

@jkotas Anything else you wanted to look at?

@jkotasjkotas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@agocke
agocke merged commit 0319010 into dotnet:mainMar 17, 2022
@hez2010
hez2010 deleted the nativeaot/sym-interface branch March 19, 2022 16:34
radekdoulik pushed a commit to radekdoulik/runtime that referenced this pull request Mar 30, 2022
* Implement missing SymUnmanaged* interfaces
* Add try-finally around GCHandle
* Marshal interface argument to pinned pointer
* Minor fixes
* Address feedbacks from review
@ghostghost locked as resolved and limited conversation to collaborators Apr 18, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-NativeAOT-coreclrcommunity-contributionIndicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cannot use BenchmarkDotNet with NativeAOT

3 participants

@hez2010@agocke@jkotas