Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Adding support for more generic dictionary entry slots for R2R - #6291

Merged
fadimounir merged 1 commit into
dotnet:masterfrom
fadimounir:methodhandle
Aug 4, 2016
Merged

fadimounir merged 1 commit into
dotnet:masterfrom
fadimounir:methodhandle

Conversation

@fadimounir

Copy link
Copy Markdown

Adding support to the following generic dictionary entry slots for R2R:
DeclaringTypeHandleSlot
ConstrainedMethodEntrySlot

@fadimounir

Copy link
Copy Markdown
Author

@JohnChen0 @jkotas PTAL

Comment thread src/inc/readytorun.h

READYTORUN_FIXUP_DelegateCtor = 0x2C, /* optimized delegate ctor */
READYTORUN_FIXUP_DeclaringTypeHandle = 0x2D,
};

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.

Do we need to bump the version number as was discussed?

@jkotas

jkotas commented Jul 17, 2016

Copy link
Copy Markdown
Member

Any interesting test cases that should be added to the R2R unit test to exercise this?

@jkotas

jkotas commented Jul 17, 2016

Copy link
Copy Markdown
Member

LGTM otherwise.

@fadimounir

Copy link
Copy Markdown
Author

@jkotas I am planning to merge these changes after merging the FieldDescHandle changes, which include the version change (see other PR).
For tests, I am considering adding a small collection of TypeGenerator tests, which cover interesting areas with generics, including the changes in this PR. I will do that once I get back from vacation

@fadimounir
fadimounir force-pushed the methodhandle branch 2 times, most recently from 60ef4d5 to 27d8c01 Compare August 4, 2016 17:33
	DeclaringTypeHandleSlot
	ConstrainedMethodEntrySlot
@fadimounir

Copy link
Copy Markdown
Author

@dotnet-bot test Windows_NT Checked pri1r2r please

@fadimounir

Copy link
Copy Markdown
Author

@dotnet-bot test Ubuntu Checked pri1r2r please

@fadimounir
fadimounir merged commit 010f42a into dotnet:master Aug 4, 2016
@fadimounir
fadimounir deleted the methodhandle branch August 16, 2016 17:59
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
…R: (dotnet/coreclr#6291)

DeclaringTypeHandleSlot
	ConstrainedMethodEntrySlot

Commit migrated from dotnet/coreclr@010f42a
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants