Skip to content

Consider types with ref fields as managed#63209

Merged
jaredpar merged 2 commits intodotnet:mainfrom
jcouv:managed-types
Aug 9, 2022
Merged

Consider types with ref fields as managed#63209
jaredpar merged 2 commits intodotnet:mainfrom
jcouv:managed-types

Conversation

@jcouv
Copy link
Member

@jcouv jcouv commented Aug 4, 2022

Fixes #63104

Relates to test plan #59194

@jcouv jcouv self-assigned this Aug 4, 2022
@ghost ghost added the Area-Compilers label Aug 4, 2022
@jcouv jcouv marked this pull request as ready for review August 5, 2022 15:43
@jcouv jcouv requested a review from a team as a code owner August 5, 2022 15:43
@jcouv jcouv requested a review from cston August 5, 2022 15:43
@jaredpar
Copy link
Member

jaredpar commented Aug 9, 2022

@cston feedback addressed

@jaredpar jaredpar enabled auto-merge (squash) August 9, 2022 16:45
@jaredpar jaredpar merged commit e3985ed into dotnet:main Aug 9, 2022
@ghost ghost added this to the Next milestone Aug 9, 2022
cston added a commit to cston/roslyn that referenced this pull request Aug 12, 2022
cston added a commit that referenced this pull request Aug 12, 2022
@dibarbet dibarbet modified the milestones: Next, 17.4 P2 Sep 1, 2022
@jcouv jcouv deleted the managed-types branch September 12, 2022 17:36
jcouv pushed a commit to jcouv/roslyn that referenced this pull request Sep 15, 2022
jcouv pushed a commit to jcouv/roslyn that referenced this pull request Sep 16, 2022
jcouv pushed a commit to jcouv/roslyn that referenced this pull request Sep 16, 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.

stackalloc should be disallowed for structs with ref fields

4 participants