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

fix anon recd creation bug (#6434) #6619

Merged
merged 3 commits into from
Apr 24, 2019
Merged

fix anon recd creation bug (#6434) #6619

merged 3 commits into from
Apr 24, 2019

Conversation

dsyme
Copy link
Contributor

@dsyme dsyme commented Apr 23, 2019

Fixes #6434

We were missing the creation of anonymous records if they didn't appear in a type signature anywhere

Also make anonymous record generation slightly simpler and more efficient.

@dsyme
Copy link
Contributor Author

dsyme commented Apr 23, 2019

Also fixes #6572

@cartermp These can be included as part of a roll up of anon record fixes along with #6619 and #6608

@dsyme dsyme added the Ready label Apr 23, 2019
@KevinRansom KevinRansom merged commit 4841e2d into dotnet:master Apr 24, 2019
@isaacabraham
Copy link
Contributor

@cartermp Do you know which version of .NET Core this fix was rolled out with?

@cartermp
Copy link
Contributor

cartermp commented Sep 5, 2019

@isaacabraham It's available in the latest preview, but other than that, no. We don't manually insert our compiler into the SDK anymore, so the answer to questions like this is generally, "when it undergoes automatic dependency update".

@KevinRansom
Copy link
Member

It is in branch 16.3, so the preview for VS2019.3 should have it:
You can download it from here:
https://visualstudio.microsoft.com/vs/preview/

@isaacabraham
Copy link
Contributor

Thanks both. I'm asking because I have that fix but someone else on the team doesn't - I have latest VS2019 update and NET Core 3 as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Compiler crashes if an anonymous record passed as obj argument
5 participants