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

[net6] Bump DotNetPreview version to 6.0.100-preview.1.21109.8 #5610

Merged
merged 12 commits into from
Mar 1, 2021

Conversation

radekdoulik
Copy link
Member

@radekdoulik radekdoulik commented Feb 10, 2021

Bump DotNetPreview version to 6.0.100-preview.1.21109.8

Add linker reflection based workaround, till
the new API is ready to use.
It forces the ILLink to process all the referenced assemblies in our steps,
without it, no assemblies are processed before MarkStep.

Updated apk size reference files. The NET6 linker has more optimizations
enabled, assembly sizes decreased about 0.5% and native shared libraries
are smaller as well - about 0.7%.

Comparison of BuildReleaseArm64 apk test sizes before/after

Simple XA NET6

Summary:
  +           0 Other entries 0.00% (of 52,514)
  +           0 Dalvik executables 0.00% (of 316,876)
  -       3,237 Assemblies -0.47% (of 690,451)
  -      34,992 Shared libraries -0.67% (of 5,202,312)
  -       7,680 Uncompressed assemblies -0.57% (of 1,357,824)
  -      10,670 Package size difference -0.36% (of 2,927,336)

XForms XA NET6

Summary:
  +           0 Other entries 0.00% (of 750,947)
  +           0 Dalvik executables 0.00% (of 3,454,884)
  -      29,117 Assemblies -0.51% (of 5,696,363)
  -      34,992 Shared libraries -0.66% (of 5,277,504)
  -      66,048 Uncompressed assemblies -0.51% (of 12,839,424)
  -      36,556 Package size difference -0.37% (of 9,899,330)

Simple XA legacy monodroid

Summary:
  +           0 Other entries 0.00% (of 52,485)
  +           0 Dalvik executables 0.00% (of 316,848)
  +           1 Assemblies 0.00% (of 1,153,531)
  +           0 Shared libraries 0.00% (of 10,510,880)
  +           0 Uncompressed assemblies 0.00% (of 2,765,312)
  -         116 Package size difference -0.00% (of 5,001,423)

XForms XA legacy monodroid

Summary:
  +           0 Other entries 0.00% (of 750,918)
  +           0 Dalvik executables 0.00% (of 3,454,264)
  -           2 Assemblies -0.00% (of 4,674,321)
  +           0 Shared libraries 0.00% (of 10,582,264)
  +           0 Uncompressed assemblies 0.00% (of 11,293,696)
  -          12 Package size difference -0.00% (of 10,476,653)

@radekdoulik
Copy link
Member Author

Testing on CI, looks like illink lost some steps.

The linker doesn't load all assemblies anymore. We depend on processing
of all assemblies before `MarkStep`.

Use reflection as temporary workaround to overcome that, until we have
new API in the linker available to handle our processing as part
of `MarkStep`.
This also gets rid of typo in previous commit, which caused problems
in XF test.
@radekdoulik radekdoulik changed the title Bump DotNetPreview version to 6.0.100-preview.1.21109.8 [net6] Bump DotNetPreview version to 6.0.100-preview.1.21109.8 Feb 25, 2021
@radekdoulik radekdoulik marked this pull request as ready for review February 25, 2021 14:53
@jonathanpeppers
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jonathanpeppers
Copy link
Member

@radekdoulik maybe we should use the same version as macios:

@radekdoulik
Copy link
Member Author

I would like to finish it with 6.0.100-preview.1.21109.8 so that I don't have to remeasure it and fix possible new issues. Will try the preview2 in new PR after that one. Hopefully it will be less bumpy than this one.

@dellis1972 dellis1972 merged commit dcd1cb7 into dotnet:master Mar 1, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Jan 24, 2024
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