Skip to content

Enable size-optimized LINQ build for Browser - #47918

Merged
stephentoub merged 2 commits into
dotnet:masterfrom
stephentoub:linqsizewasm
Feb 9, 2021
Merged

Enable size-optimized LINQ build for Browser#47918
stephentoub merged 2 commits into
dotnet:masterfrom
stephentoub:linqsizewasm

Conversation

@stephentoub

@stephentoub stephentoub commented Feb 5, 2021

Copy link
Copy Markdown
Member

Re-enable the size-optimized variant of LINQ and use it for the Browser build of the library. (This was previously added not for IL size but for AOT asm size, but it's also just less IL.)

This reduces the size of a default Blazor wasm app by ~20K uncompressed and ~7K compressed.

Fixes #45399
cc: @eerhardt, @tannergooding

@stephentoub stephentoub added area-System.Linq size-reduction Issues impacting final app size primary for size sensitive workloads labels Feb 5, 2021
@stephentoub stephentoub added this to the 6.0.0 milestone Feb 5, 2021
@ghost

ghost commented Feb 5, 2021

Copy link
Copy Markdown

Tagging subscribers to this area: @eiriktsarpalis
See info in area-owners.md if you want to be subscribed.

Issue Details

Re-enable the size-optimized variant of LINQ and use it for the Browser build of the library. (This was previously added not for IL size but for AOT asm size, but it's also just less IL.)

This reduces the size of a default Blazor wasm app by ~20K uncompressed and ~7K compressed.

cc: @eerhardt, @tannergooding

Author: stephentoub
Assignees: -
Labels:

area-System.Linq, size-reduction

Milestone: 6.0.0

@marek-safar

Copy link
Copy Markdown
Contributor

I think this is worth expanding to mobile targets (iOS, tvOS, Android) as well

Comment thread src/libraries/System.Linq/tests/EmptyPartitionTests.cs Outdated
Re-enable the size-optimized variant of LINQ and use it for the Browser build of the library.
@stephentoub
stephentoub merged commit 835f5b4 into dotnet:master Feb 9, 2021
@stephentoub
stephentoub deleted the linqsizewasm branch February 9, 2021 14:30
@ghost ghost locked as resolved and limited conversation to collaborators Mar 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-System.Linq size-reduction Issues impacting final app size primary for size sensitive workloads

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Consider using .SizeOpt.cs in System.Linq built for Browser/Xamarin/AOT

3 participants