Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move System.Runtime.CompilerServices.CallConv* types to System.Private.CoreLib and expose via System.Runtime. #38473

Merged
merged 2 commits into from
Jun 27, 2020

Conversation

jkoritzinsky
Copy link
Member

Move the calling convention types used in the new calling convention modopt-based encoding down to System.Private.CoreLib/System.Runtime so they live in the same assembly as System.Object.

Fixes #38132

…modopt-based encoding down to System.Private.CoreLib/System.Runtime.
@ghost
Copy link

ghost commented Jun 26, 2020

Hello @jkoritzinsky!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@jkotas
Copy link
Member

jkotas commented Jun 27, 2020

I have just noticed that the UnmanagedCallersOnlyAttribute is not in the System.Runtime contract. Is that fine with Roslyn, or does this attribute need to be moved as well?

cc @333fred

@333fred
Copy link
Member

333fred commented Jun 27, 2020

I have just noticed that the UnmanagedCallersOnlyAttribute is not in the System.Runtime contract. Is that fine with Roslyn, or does this attribute need to be moved as well?

I think we'd prefer it to be in corelib, but I don't think we were going to require it.

@jkotas jkotas merged commit 1b7edcb into dotnet:master Jun 27, 2020
@jkoritzinsky jkoritzinsky deleted the lower-callconv-types branch June 27, 2020 16:25
@ghost ghost locked as resolved and limited conversation to collaborators Dec 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move CallConv* to the same reference assembly as System.Object
5 participants