Skip to content

Commit 9b0c324

Browse files
jonathanpeppersjonathanpeppers
committed
[One .NET] AOT support
Helpful reading: * https://github.com/dotnet/runtime/blob/15dec9a2aa5a4236d6ba70de2e9c146867b9d2e0/src/tasks/AotCompilerTask/MonoAOTCompiler.cs * https://github.com/dotnet/runtime/blob/15dec9a2aa5a4236d6ba70de2e9c146867b9d2e0/src/mono/netcore/nuget/Microsoft.NET.Runtime.MonoAOTCompiler.Task/README.md To make this work, I moved the existing `<Aot/>` MSBuild task calls to a new `_AndroidAot` MSBuild target in `Xamarin.Android.Legacy.targets`. In the .NET 6 targets, there is a *different* `_AndroidAot` MSBuild target that runs the new `<MonoAOTCompiler/>` MSBuild task. The `_AndroidAot` target runs per `$(RuntimeIdentifier)` after the linker completes. Native libraries are added to the `@(ResolvedFileToPublish)` item group to be included in the final `.apk`. To follow convention with Blazor WASM: https://devblogs.microsoft.com/aspnet/asp-net-core-updates-in-net-6-preview-4/#blazor-webassembly-ahead-of-time-aot-compilation The `$(RunAOTCompilation)` MSBuild property enables AOT. To help with backwards compatibility with "legacy" Xamarin.Android, I kept the `$(AotAssemblies)` MSBuild property intact. ~~ Results ~~ All tests were running on a Pixel 5. Using the HelloAndroid app: https://github.com/dotnet/maui-samples/tree/main/HelloAndroid Defaults to two architectures: arm64 and x86 Average of 10 runs with `-c Release` and no AOT: Activity: Displayed 00:00:00.308 Apk size: 8367969 Average of 10 runs with `-c Release -p:RunAOTCompilation=true`: Activity: Displayed 00:00:00.209 Apk size: 12082123 Using the HelloMaui app: https://github.com/dotnet/maui-samples/tree/main/HelloMaui Defaults to two architectures: arm64 and x86 Average of 10 runs with `-c Release` and no AOT: Activity: Displayed 00:00:01.117 Apk size: 16272964 Average of 10 runs with `-c Release -p:RunAOTCompilation=true`: Activity: Displayed 00:00:00.568 Apk size: 42869016
1 parent 1e5bfa3 commit 9b0c324

File tree

14 files changed

+469
-269
lines changed

14 files changed

+469
-269
lines changed

‎Documentation/guides/OneDotNet.md‎

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -164,6 +164,18 @@ It is recommended to migrate to the new linker settings, as
164164
[linker]: https://docs.microsoft.com/dotnet/core/deploying/trimming-options
165165
[linker-full]: https://docs.microsoft.com/dotnet/core/deploying/trimming-options#trimmed-assemblies
166166

167+
## AOT
168+
169+
`$(RunAOTCompilation)` will be the new MSBuild property for enabling
170+
AOT. This is the same property used for [Blazor WASM][blazor].
171+
`$(AotAssemblies)` will also enable AOT, in order to help with
172+
migration from "legacy" Xamarin.Android to .NET 6.
173+
174+
It is recommended to migrate to the new `$(RunAOTCompilation)`
175+
property, as `$(AotAssemblies)` will eventually be deprecated.
176+
177+
[blazor]: https://devblogs.microsoft.com/aspnet/asp-net-core-updates-in-net-6-preview-4/#blazor-webassembly-ahead-of-time-aot-compilation
178+
167179
## dotnet cli
168180

169181
There are currently a few "verbs" we are aiming to get working in

‎build-tools/automation/azure-pipelines.yaml‎

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ variables:
4949
# - This is a non-fork branch with name containing "mono-" (for Mono bumps)
5050
IsMonoBranch: $[and(ne(variables['System.PullRequest.IsFork'], 'True'), or(contains(variables['Build.SourceBranchName'], 'mono-'), contains(variables['System.PullRequest.SourceBranch'], 'mono-')))]
5151
RunAllTests: $[or(eq(variables['XA.RunAllTests'], true), eq(variables['IsMonoBranch'], true))]
52-
DotNetNUnitCategories: '& TestCategory != DotNetIgnore & TestCategory != AOT & TestCategory != MkBundle & TestCategory != MonoSymbolicate & TestCategory != PackagesConfig & TestCategory != StaticProject & TestCategory != Debugger & TestCategory != SystemApplication'
52+
DotNetNUnitCategories: '& TestCategory != DotNetIgnore & TestCategory != HybridAOT & TestCategory != ProfiledAOT & TestCategory != MkBundle & TestCategory != MonoSymbolicate & TestCategory != PackagesConfig & TestCategory != StaticProject & TestCategory != Debugger & TestCategory != SystemApplication'
5353
NUnit.NumberOfTestWorkers: 4
5454
GitHub.Token: $(github--pat--vs-mobiletools-engineering-service2)
5555
CONVERT_JAVADOC_TO_XMLDOC: $[ne(variables['Build.DefinitionName'], 'Xamarin.Android-PR')]
@@ -798,6 +798,17 @@ stages:
798798
artifactFolder: net6-Interpreter
799799
useDotNet: true
800800

801+
- template: yaml-templates/apk-instrumentation.yaml
802+
parameters:
803+
configuration: $(XA.Build.Configuration)
804+
testName: Mono.Android.NET_Tests-Aot
805+
project: tests/Mono.Android-Tests/Runtime-Microsoft.Android.Sdk/Mono.Android.NET-Tests.csproj
806+
testResultsFiles: TestResult-Mono.Android.NET_Tests-$(XA.Build.Configuration)-Aot.xml
807+
extraBuildArgs: /p:RunAOTCompilation=true
808+
artifactSource: bin/Test$(XA.Build.Configuration)/net6.0-android/Mono.Android.NET_Tests-Signed.apk
809+
artifactFolder: net6-aot
810+
useDotNet: true
811+
801812
- task: MSBuild@1
802813
displayName: shut down emulator
803814
inputs:

‎src/Xamarin.Android.Build.Tasks/Microsoft.Android.Sdk/targets/Microsoft.Android.Sdk.After.targets‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ This file is imported *after* the Microsoft.NET.Sdk/Sdk.targets.
1818
<Import Project="..\tools\Xamarin.Android.Bindings.Core.targets" />
1919
<Import Project="..\tools\Xamarin.Android.Bindings.ClassParse.targets" />
2020
<Import Project="Microsoft.Android.Sdk.AndroidLibraries.targets" />
21+
<Import Project="Microsoft.Android.Sdk.Aot.targets" Condition=" '$(AndroidApplication)' == 'true' " />
2122
<Import Project="Microsoft.Android.Sdk.Application.targets" Condition=" '$(AndroidApplication)' == 'true' " />
2223
<Import Project="Microsoft.Android.Sdk.AssemblyResolution.targets" />
2324
<Import Project="Microsoft.Android.Sdk.ILLink.targets" />

‎src/Xamarin.Android.Build.Tasks/Microsoft.Android.Sdk/targets/Microsoft.Android.Sdk.Aot.targets‎

Lines changed: 72 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,72 @@
1+
<!--
2+
***********************************************************************************************
3+
Microsoft.Android.Sdk.Aot.targets
4+
5+
.NET 6 AOT support. You can find "legacy" Xamarin.Android AOT support
6+
in Xamarin.Android.Legacy.targets.
7+
8+
For <MonoAOTCompiler/> usage, see:
9+
* https://github.com/dotnet/runtime/blob/15dec9a2aa5a4236d6ba70de2e9c146867b9d2e0/src/tasks/AotCompilerTask/MonoAOTCompiler.cs
10+
* https://github.com/dotnet/runtime/blob/15dec9a2aa5a4236d6ba70de2e9c146867b9d2e0/src/mono/netcore/nuget/Microsoft.NET.Runtime.MonoAOTCompiler.Task/README.md
11+
12+
These targets are running within the _ComputeFilesToPublishForRuntimeIdentifiers target.
13+
They run in a context of an inner build with a single $(RuntimeIdentifier).
14+
15+
***********************************************************************************************
16+
-->
17+
<Project>
18+
19+
<UsingTask TaskName="Xamarin.Android.Tasks.GetAotArguments" AssemblyFile="$(_XamarinAndroidBuildTasksAssembly)" />
20+
21+
<Target Name="_AndroidAotInputs">
22+
<ItemGroup>
23+
<_AndroidAotInputs Include="@(ResolvedFileToPublish)" Condition=" '%(Extension)' == '.dll' " />
24+
</ItemGroup>
25+
</Target>
26+
27+
<Target Name="_AndroidAot"
28+
Condition=" '$(AotAssemblies)' == 'true' and '$(RuntimeIdentifier)' != '' "
29+
DependsOnTargets="_AndroidAotInputs"
30+
Inputs="@(_AndroidAotInputs)"
31+
Outputs="$(_AndroidStampDirectory)_AndroidAot.stamp">
32+
<GetAotArguments
33+
AndroidAotMode="$(AndroidAotMode)"
34+
AndroidNdkDirectory="$(_AndroidNdkDirectory)"
35+
AndroidBinUtilsDirectory="$(AndroidBinUtilsDirectory)"
36+
AndroidApiLevel="$(_AndroidApiLevel)"
37+
ManifestFile="$(_OuterIntermediateOutputPath)android\AndroidManifest.xml"
38+
AndroidSequencePointsMode="$(_SequencePointsMode)"
39+
AotAdditionalArguments="$(AndroidAotAdditionalArguments)"
40+
AotOutputDirectory="$(_AndroidAotBinDirectory)"
41+
RuntimeIdentifier="$(RuntimeIdentifier)"
42+
EnableLLVM="$(EnableLLVM)"
43+
Profiles="@(_AotProfiles)">
44+
<Output PropertyName="_AotArguments" TaskParameter="Arguments" />
45+
</GetAotArguments>
46+
<ItemGroup>
47+
<_MonoAOTAssemblies Include="@(_AndroidAotInputs->'%(FullPath)')" AotArguments="$(_AotArguments)" />
48+
</ItemGroup>
49+
<MakeDir Directories="$(IntermediateOutputPath)aot\" />
50+
<MonoAOTCompiler
51+
Assemblies="@(_MonoAOTAssemblies)"
52+
CompilerBinaryPath="@(MonoAotCrossCompiler->WithMetadataValue('RuntimeIdentifier', '$(RuntimeIdentifier)'))"
53+
DisableParallelAot="$(_DisableParallelAot)"
54+
LibraryFormat="So"
55+
LLVMPath="$(_LLVMPath)"
56+
Mode="$(AndroidAotMode)"
57+
OutputDir="$(IntermediateOutputPath)aot\"
58+
OutputType="Library"
59+
UseAotDataFile="false"
60+
UseLLVM="$(EnableLLVM)">
61+
<Output TaskParameter="CompiledAssemblies" ItemName="_AotCompiledAssemblies" />
62+
<Output TaskParameter="FileWrites" ItemName="FileWrites" />
63+
</MonoAOTCompiler>
64+
<Touch Files="$(_AndroidStampDirectory)_AndroidAot.stamp" AlwaysCreate="true" />
65+
<ItemGroup>
66+
<ResolvedFileToPublish
67+
Include="@(_AotCompiledAssemblies->'%(LibraryFile)')"
68+
ArchiveFileName="libaot-$([System.IO.Path]::GetFileNameWithoutExtension('%(_AotCompiledAssemblies.LibraryFile)')).so"
69+
/>
70+
</ItemGroup>
71+
</Target>
72+
</Project>

‎src/Xamarin.Android.Build.Tasks/Microsoft.Android.Sdk/targets/Microsoft.Android.Sdk.AssemblyResolution.targets‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ _ResolveAssemblies MSBuild target.
3131
</PropertyGroup>
3232

3333
<Target Name="_ComputeFilesToPublishForRuntimeIdentifiers"
34-
DependsOnTargets="_FixupIntermediateAssembly;ResolveReferences;ComputeFilesToPublish"
34+
DependsOnTargets="_FixupIntermediateAssembly;ResolveReferences;ComputeFilesToPublish;_AndroidAot"
3535
Returns="@(ResolvedFileToPublish)">
3636
<ItemGroup>
3737
<ResolvedFileToPublish

‎src/Xamarin.Android.Build.Tasks/Microsoft.Android.Sdk/targets/Microsoft.Android.Sdk.DefaultProperties.targets‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,9 @@
6868
<AndroidManifest Condition=" '$(AndroidManifest)' == '' and Exists ('Properties\AndroidManifest.xml') and !Exists ('AndroidManifest.xml') ">Properties\AndroidManifest.xml</AndroidManifest>
6969
<AndroidManifest Condition=" '$(AndroidManifest)' == '' ">AndroidManifest.xml</AndroidManifest>
7070
<GenerateApplicationManifest Condition=" '$(GenerateApplicationManifest)' == '' ">true</GenerateApplicationManifest>
71+
<RunAOTCompilation Condition=" '$(RunAOTCompilation)' == '' and '$(AotAssemblies)' == 'true' ">true</RunAOTCompilation>
72+
<RunAOTCompilation Condition=" '$(RunAOTCompilation)' == '' ">false</RunAOTCompilation>
73+
<AotAssemblies>$(RunAOTCompilation)</AotAssemblies>
7174

7275
<!--
7376
Runtime libraries feature switches defaults

0 commit comments

Comments
 (0)