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

Retarget to .NET 5.0 #273

Merged
merged 9 commits into from Jan 26, 2022
Merged

Retarget to .NET 5.0 #273

merged 9 commits into from Jan 26, 2022

Conversation

kant2002
Copy link
Contributor

@kant2002 kant2002 commented Aug 24, 2020

Update to .NET 5.0
This brings a lot of performance improvements to the solution.

@kant2002 kant2002 force-pushed the kant/net50 branch 3 times, most recently from d1811ea to aec9f9d Compare September 1, 2020 15:35
@kant2002 kant2002 mentioned this pull request Sep 1, 2020
6 tasks
Base automatically changed from master to main March 17, 2021 19:19
@kant2002
Copy link
Contributor Author

/cc @tminka

@gfoidl
Copy link
Member

gfoidl commented Nov 20, 2021

Could be updated to .NET 6 now too, to get even more perf goodness for (almost) free.

@tminka
Copy link
Contributor

tminka commented Jan 26, 2022

Please update the branch and re-run the checks. They should pass now.

@kant2002
Copy link
Contributor Author

Please update the branch and re-run the checks. They should pass now.

Gladly.

@kant2002
Copy link
Contributor Author

@tminka while I have a bit of your attention, can I ask couple questions?

  • Should I update this PR to .NET 6 given that it would be LTS version
  • What do you think about support for AOT compilation in this library? I know that this lib uses System.Reflection.Emit which is not very AOT friendly, but maybe there other options which can improve current state of things. It would be unreasonable for me to try improve things without your support.

@tminka
Copy link
Contributor

tminka commented Jan 26, 2022

It seems best to have one commit that corresponds to .NET 5, and another one that corresponds to .NET 6, so that it is possible for anyone to get the appropriate version from the git history.

I'm not familiar with what is needed for AOT. There haven't been any requests for it.

@kant2002
Copy link
Contributor Author

Thanks!! I was looking at this project and cannot spot the issue

@tminka tminka merged commit 1b4c479 into dotnet:main Jan 26, 2022
@kant2002 kant2002 deleted the kant/net50 branch January 26, 2022 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants