Skip to content

Conversation

@tmat
Copy link
Member

@tmat tmat commented Aug 3, 2022

The attribute, if defined, is emitted into EnC deltas on reloadable types.

Fixes #59954

@tmat tmat requested a review from a team as a code owner August 3, 2022 18:01
@ghost ghost added the Area-Compilers label Aug 3, 2022
@tmat
Copy link
Member Author

tmat commented Aug 3, 2022

@davidwengier @cston @jcouv PTAL

@tmat tmat added this to the 17.4 milestone Aug 3, 2022
Copy link
Member

@jcouv jcouv left a comment

Choose a reason for hiding this comment

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

LGTM Thanks (iteration 2) but a couple of questions to clarify or consider

Copy link
Member

@jcouv jcouv left a comment

Choose a reason for hiding this comment

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

LGTM Thanks (iteration 3)

@tmat tmat enabled auto-merge (squash) August 4, 2022 16:31
@tmat tmat force-pushed the OriginalTypeAttr branch from 6ca7ef9 to e2541b2 Compare August 6, 2022 01:41
@tmat tmat merged commit 550f811 into dotnet:main Aug 6, 2022
@ghost ghost modified the milestones: 17.4, Next Aug 6, 2022
@dibarbet dibarbet modified the milestones: Next, 17.4 P2 Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add an attribute specifying the original type, when we create a diff of a ReloadableType

5 participants