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

Emit/Call member references in new ILGenerator #93564

Closed
wants to merge 8 commits into from

Conversation

buyaa-n
Copy link
Member

@buyaa-n buyaa-n commented Oct 16, 2023

Add implementation for Emit(OpCode opcode, FieldInfo field), Emit(OpCode opcode, MethodInfo meth), Emit(OpCode opcode, Type cls) and EmitCall(OpCode opcode, MethodInfo methodInfo, Type[]? optionalParameterTypes)

The PR depend on #93244 which is blocked by #93419, will be cleaned up after #93244 merged.
Contributes to #92975

@ghost
Copy link

ghost commented Oct 16, 2023

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

Issue Details

Add implementation for Emit(OpCode opcode, FieldInfo field), Emit(OpCode opcode, MethodInfo meth), Emit(OpCode opcode, Type cls) and EmitCall(OpCode opcode, MethodInfo methodInfo, Type[]? optionalParameterTypes)

The PR depend on #93244, will be cleaned up after #93244 merged. Also blocked by #93497.

Contributes to #92975

Author: buyaa-n
Assignees: -
Labels:

area-System.Reflection.Emit

Milestone: -

@dotnet-issue-labeler
Copy link

Note regarding the new-api-needs-documentation label:

This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change.

@buyaa-n
Copy link
Member Author

buyaa-n commented Oct 27, 2023

Created a new PR

@buyaa-n buyaa-n closed this Oct 27, 2023
@buyaa-n buyaa-n deleted the call-members branch October 27, 2023 21:50
@ghost ghost locked as resolved and limited conversation to collaborators Nov 27, 2023
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.

None yet

1 participant