Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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 dotnet#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 dotnet#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 dotnet#12147.
bad799d 
Fixes dotnet#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
# for free to subscribe to this conversation on GitHub. Already have an account? #.
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