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.

Sign up for GitHub

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

Jump to bottom

.NET: make llvm the default for release builds #12147

Closed
rolfbjarne opened this issue Jul 20, 2021 · 1 comment · Fixed by #12660
Closed

.NET: make llvm the default for release builds #12147

rolfbjarne opened this issue Jul 20, 2021 · 1 comment · Fixed by #12660
Assignees
@rolfbjarne
Labels
dotnet An issue or pull request related to .NET (6) dotnet-pri0 .NET 6: required for stable release
Milestone
.NET 6

Comments

@rolfbjarne
Copy link
Member

rolfbjarne commented Jul 20, 2021 •
edited
Loading

As the title says.

A "release build" is defined to be a build where "Configuration=Release".

Blocked on dotnet/runtime#56100.
@rolfbjarne rolfbjarne added dotnet An issue or pull request related to .NET (6) dotnet-pri0 .NET 6: required for stable release labels Jul 20, 2021
@rolfbjarne rolfbjarne added this to the .NET 6 milestone Jul 20, 2021
@rolfbjarne rolfbjarne mentioned this issue Jul 20, 2021
Merged
spouliot pushed a commit to spouliot/xamarin-macios that referenced this issue Jul 28, 2021
[test][dotnet] Manually enable LLVM for app size comparison
21dd6c6 
without LLVM https://gist.github.com/spouliot/a1fb4ad87b98c3bd0e59a0501b243e09
with LLVM https://gist.github.com/spouliot/f6e8281c8b546b56991777ad1e919e4d

This should become the default xamarin#12147
so this commit is only part of `preview.7` (and not `main`)
@spouliot spouliot mentioned this issue Jul 28, 2021
Merged
spouliot added a commit that referenced this issue Jul 28, 2021
@spouliot
[test][dotnet] Manually enable LLVM for app size comparison (#12271)
e685f63 
without LLVM https://gist.github.com/spouliot/a1fb4ad87b98c3bd0e59a0501b243e09
with LLVM https://gist.github.com/spouliot/f6e8281c8b546b56991777ad1e919e4d

This should become the default #12147
so this commit is only part of `preview.7` (and not `main`)
@rolfbjarne rolfbjarne self-assigned this Jul 29, 2021
vs-mobiletools-engineering-service2 pushed a commit to vs-mobiletools-engineering-service2/xamarin-macios that referenced this issue Aug 19, 2021
[test][dotnet] Manually enable LLVM for app size comparison
8cf7e48 
without LLVM https://gist.github.com/spouliot/a1fb4ad87b98c3bd0e59a0501b243e09
with LLVM https://gist.github.com/spouliot/f6e8281c8b546b56991777ad1e919e4d

This should become the default xamarin#12147
so this commit is only part of `preview.7` (and not `main`)
Merged
spouliot pushed a commit that referenced this issue Aug 23, 2021
@vs-mobiletools-engineering-service2
[test][dotnet] Manually enable LLVM for app size comparison (#12487)
63d8a5a 
without LLVM https://gist.github.com/spouliot/a1fb4ad87b98c3bd0e59a0501b243e09
with LLVM https://gist.github.com/spouliot/f6e8281c8b546b56991777ad1e919e4d

This should become the default #12147
so this commit is only part of `preview.7` (and not `main`)
@spouliot
Copy link
Contributor

#12546 makes it hard to set LLVM to default since any fat application won't compile (until it's fixed)

@spouliot spouliot mentioned this issue Sep 7, 2021
Open
6 tasks
rolfbjarne added a commit to rolfbjarne/xamarin-macios that referenced this issue Sep 8, 2021
@rolfbjarne
[dotnet] Enable LLVM by default for release builds. Fixes xamarin#12147.
bad799d 
Fixes xamarin#12147.
@rolfbjarne rolfbjarne mentioned this issue Sep 8, 2021
Merged
rolfbjarne added a commit that referenced this issue Sep 8, 2021
@rolfbjarne
[dotnet] Enable LLVM by default for release builds. Fixes #12147. (#1…
c2fda6e 
…2660)

Fixes #12147.
@ghost ghost locked as resolved and limited conversation to collaborators Apr 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@rolfbjarne rolfbjarne

Labels
dotnet An issue or pull request related to .NET (6) dotnet-pri0 .NET 6: required for stable release
Projects
None yet
Milestone
.NET 6
Development

Successfully merging a pull request may close this issue.

[dotnet] Enable LLVM by default for release builds. Fixes #12147. rolfbjarne/xamarin-macios
2 participants
@rolfbjarne @spouliot