-
Notifications
You must be signed in to change notification settings - Fork 561
[net11.0] Support multiple R2R images for ios/tvos apps #25077
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
Open
rolfbjarne
wants to merge
10
commits into
net11.0
Choose a base branch
from
dev/rolf/dev/jekoritz/r2r-multiple-images
base: net11.0
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
10 commits
Select commit
Hold shift + click to select a range
af34218
[R2R] Support multiple R2R modules with per-module frameworks
jkoritzinsky f2445b2
Add FrameworkName metadata for R2R post-processing
rolfbjarne 1e817d0
Update sizes
rolfbjarne a926910
Merge remote-tracking branch 'origin/net11.0' into dev/rolf/dev/jekor…
rolfbjarne df29933
[tests] Update expected sizes.
rolfbjarne 955b20c
Address PR feedback: remove fallback, fix incremental builds, add val…
jkoritzinsky 8cea1fd
Fix build failure and update expected sizes
jkoritzinsky 96ec7b6
Use full dylib filename as R2R module name
jkoritzinsky a1aec4b
Update expected sizes
jkoritzinsky 550f56d
Support Windows remote builds for R2R module registration
jkoritzinsky File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
89 changes: 89 additions & 0 deletions
89
msbuild/Xamarin.MacDev.Tasks/Tasks/GenerateR2RModuleRegistration.cs
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,89 @@ | ||
| using System.IO; | ||
| using System.Linq; | ||
| using System.Collections.Generic; | ||
| using System.Text; | ||
|
|
||
| using Microsoft.Build.Framework; | ||
| using Microsoft.Build.Utilities; | ||
|
|
||
| using Xamarin.Messaging.Build.Client; | ||
|
|
||
| #nullable enable | ||
|
|
||
| namespace Xamarin.MacDev.Tasks { | ||
| public class GenerateR2RModuleRegistration : XamarinTask, ITaskCallback { | ||
|
|
||
| #region Inputs | ||
| [Required] | ||
| public ITaskItem [] R2RModules { get; set; } = []; | ||
|
|
||
| [Required] | ||
| public string OutputFile { get; set; } = ""; | ||
| #endregion | ||
|
|
||
| public override bool Execute () | ||
| { | ||
| if (ShouldExecuteRemotely ()) | ||
| return ExecuteRemotely (); | ||
|
|
||
| var sb = new StringBuilder (); | ||
|
|
||
| sb.AppendLine ("#include \"xamarin/xamarin.h\""); | ||
| sb.AppendLine (); | ||
|
|
||
| foreach (var module in R2RModules) { | ||
| var symbolName = module.GetMetadata ("SymbolName"); | ||
| if (string.IsNullOrEmpty (symbolName)) { | ||
| Log.LogError ("Missing '{0}' metadata on item '{1}'.", "SymbolName", module.ItemSpec); | ||
| continue; | ||
| } | ||
| sb.AppendLine ($"extern void* {symbolName};"); | ||
| } | ||
|
|
||
| sb.AppendLine (); | ||
| sb.AppendLine ("static struct xamarin_r2r_module r2r_module_entries [] = {"); | ||
|
|
||
| foreach (var module in R2RModules) { | ||
| var moduleName = module.GetMetadata ("ModuleName"); | ||
| var symbolName = module.GetMetadata ("SymbolName"); | ||
| if (string.IsNullOrEmpty (moduleName)) { | ||
| Log.LogError ("Missing '{0}' metadata on item '{1}'.", "ModuleName", module.ItemSpec); | ||
| continue; | ||
| } | ||
| if (string.IsNullOrEmpty (symbolName)) | ||
| continue; // already reported above | ||
| var escapedModuleName = moduleName.Replace ("\\", "\\\\").Replace ("\"", "\\\""); | ||
| sb.AppendLine ($"\t{{ \"{escapedModuleName}\", &{symbolName} }},"); | ||
| } | ||
|
|
||
| sb.AppendLine ("};"); | ||
| sb.AppendLine (); | ||
|
|
||
| sb.AppendLine ("__attribute__ ((constructor))"); | ||
| sb.AppendLine ("static void xamarin_register_r2r_modules ()"); | ||
| sb.AppendLine ("{"); | ||
| sb.AppendLine ("\txamarin_r2r_modules = r2r_module_entries;"); | ||
| sb.AppendLine ("\txamarin_r2r_module_count = sizeof (r2r_module_entries) / sizeof (r2r_module_entries [0]);"); | ||
| sb.AppendLine ("}"); | ||
|
|
||
| if (Log.HasLoggedErrors) | ||
| return false; | ||
|
|
||
| var content = sb.ToString (); | ||
| var outputDir = Path.GetDirectoryName (OutputFile); | ||
| if (!string.IsNullOrEmpty (outputDir)) | ||
| Directory.CreateDirectory (outputDir); | ||
|
|
||
| if (!File.Exists (OutputFile) || File.ReadAllText (OutputFile) != content) | ||
| File.WriteAllText (OutputFile, content); | ||
|
|
||
| return !Log.HasLoggedErrors; | ||
| } | ||
|
|
||
| public bool ShouldCopyToBuildServer (ITaskItem item) => false; | ||
|
|
||
| public bool ShouldCreateOutputFile (ITaskItem item) => true; | ||
|
|
||
| public IEnumerable<ITaskItem> GetAdditionalItemsToBeCopied () => Enumerable.Empty<ITaskItem> (); | ||
| } | ||
| } | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
28 changes: 14 additions & 14 deletions
28
tests/dotnet/UnitTests/expected/iOS-CoreCLR-Interpreter-size.txt
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,36 +1,36 @@ | ||
| AppBundleSize: 9,269,302 bytes (9,052.1 KB = 8.8 MB) | ||
| AppBundleSize: 9,279,285 bytes (9,061.8 KB = 8.8 MB) | ||
| # The following list of files and their sizes is just informational / for review, and isn't used in the test: | ||
| _CodeSignature/CodeResources: 10,847 bytes (10.6 KB = 0.0 MB) | ||
| archived-expanded-entitlements.xcent: 384 bytes (0.4 KB = 0.0 MB) | ||
| Frameworks/libcoreclr.framework/_CodeSignature/CodeResources: 1,798 bytes (1.8 KB = 0.0 MB) | ||
| Frameworks/libcoreclr.framework/Info.plist: 840 bytes (0.8 KB = 0.0 MB) | ||
| Frameworks/libcoreclr.framework/Info.plist: 833 bytes (0.8 KB = 0.0 MB) | ||
| Frameworks/libcoreclr.framework/libcoreclr: 5,220,544 bytes (5,098.2 KB = 5.0 MB) | ||
| Frameworks/libSystem.Globalization.Native.framework/_CodeSignature/CodeResources: 1,798 bytes (1.8 KB = 0.0 MB) | ||
| Frameworks/libSystem.Globalization.Native.framework/Info.plist: 882 bytes (0.9 KB = 0.0 MB) | ||
| Frameworks/libSystem.Globalization.Native.framework/libSystem.Globalization.Native: 109,232 bytes (106.7 KB = 0.1 MB) | ||
| Frameworks/libSystem.Globalization.Native.framework/Info.plist: 875 bytes (0.9 KB = 0.0 MB) | ||
| Frameworks/libSystem.Globalization.Native.framework/libSystem.Globalization.Native: 109,248 bytes (106.7 KB = 0.1 MB) | ||
| Frameworks/libSystem.IO.Compression.Native.framework/_CodeSignature/CodeResources: 1,798 bytes (1.8 KB = 0.0 MB) | ||
| Frameworks/libSystem.IO.Compression.Native.framework/Info.plist: 884 bytes (0.9 KB = 0.0 MB) | ||
| Frameworks/libSystem.IO.Compression.Native.framework/libSystem.IO.Compression.Native: 1,431,280 bytes (1,397.7 KB = 1.4 MB) | ||
| Frameworks/libSystem.IO.Compression.Native.framework/Info.plist: 877 bytes (0.9 KB = 0.0 MB) | ||
| Frameworks/libSystem.IO.Compression.Native.framework/libSystem.IO.Compression.Native: 1,431,296 bytes (1,397.8 KB = 1.4 MB) | ||
| Frameworks/libSystem.Native.framework/_CodeSignature/CodeResources: 1,798 bytes (1.8 KB = 0.0 MB) | ||
| Frameworks/libSystem.Native.framework/Info.plist: 854 bytes (0.8 KB = 0.0 MB) | ||
| Frameworks/libSystem.Native.framework/libSystem.Native: 162,128 bytes (158.3 KB = 0.2 MB) | ||
| Frameworks/libSystem.Native.framework/Info.plist: 847 bytes (0.8 KB = 0.0 MB) | ||
| Frameworks/libSystem.Native.framework/libSystem.Native: 162,224 bytes (158.4 KB = 0.2 MB) | ||
| Frameworks/libSystem.Net.Security.Native.framework/_CodeSignature/CodeResources: 1,798 bytes (1.8 KB = 0.0 MB) | ||
| Frameworks/libSystem.Net.Security.Native.framework/Info.plist: 880 bytes (0.9 KB = 0.0 MB) | ||
| Frameworks/libSystem.Net.Security.Native.framework/libSystem.Net.Security.Native: 88,000 bytes (85.9 KB = 0.1 MB) | ||
| Frameworks/libSystem.Net.Security.Native.framework/Info.plist: 873 bytes (0.9 KB = 0.0 MB) | ||
| Frameworks/libSystem.Net.Security.Native.framework/libSystem.Net.Security.Native: 88,016 bytes (86.0 KB = 0.1 MB) | ||
| Frameworks/libSystem.Security.Cryptography.Native.Apple.framework/_CodeSignature/CodeResources: 1,798 bytes (1.8 KB = 0.0 MB) | ||
| Frameworks/libSystem.Security.Cryptography.Native.Apple.framework/Info.plist: 910 bytes (0.9 KB = 0.0 MB) | ||
| Frameworks/libSystem.Security.Cryptography.Native.Apple.framework/Info.plist: 903 bytes (0.9 KB = 0.0 MB) | ||
| Frameworks/libSystem.Security.Cryptography.Native.Apple.framework/libSystem.Security.Cryptography.Native.Apple: 194,256 bytes (189.7 KB = 0.2 MB) | ||
| Info.plist: 1,168 bytes (1.1 KB = 0.0 MB) | ||
| Info.plist: 1,161 bytes (1.1 KB = 0.0 MB) | ||
| Microsoft.iOS.dll: 98,816 bytes (96.5 KB = 0.1 MB) | ||
| PkgInfo: 8 bytes (0.0 KB = 0.0 MB) | ||
| runtimeconfig.bin: 1,481 bytes (1.4 KB = 0.0 MB) | ||
| SizeTestApp: 195,856 bytes (191.3 KB = 0.2 MB) | ||
| SizeTestApp: 196,528 bytes (191.9 KB = 0.2 MB) | ||
| SizeTestApp.dll: 7,680 bytes (7.5 KB = 0.0 MB) | ||
| System.Collections.Immutable.dll: 14,848 bytes (14.5 KB = 0.0 MB) | ||
| System.Diagnostics.StackTrace.dll: 8,192 bytes (8.0 KB = 0.0 MB) | ||
| System.IO.Compression.dll: 22,528 bytes (22.0 KB = 0.0 MB) | ||
| System.IO.MemoryMappedFiles.dll: 22,016 bytes (21.5 KB = 0.0 MB) | ||
| System.Private.CoreLib.dll: 1,566,208 bytes (1,529.5 KB = 1.5 MB) | ||
| System.Private.CoreLib.dll: 1,575,424 bytes (1,538.5 KB = 1.5 MB) | ||
| System.Reflection.Metadata.dll: 84,480 bytes (82.5 KB = 0.1 MB) | ||
| System.Runtime.dll: 5,120 bytes (5.0 KB = 0.0 MB) | ||
| System.Runtime.InteropServices.dll: 8,192 bytes (8.0 KB = 0.0 MB) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
12 changes: 6 additions & 6 deletions
12
tests/dotnet/UnitTests/expected/iOS-MonoVM-interpreter-size.txt
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,14 +1,14 @@ | ||
| AppBundleSize: 3,617,184 bytes (3,532.4 KB = 3.4 MB) | ||
| AppBundleSize: 3,622,219 bytes (3,537.3 KB = 3.5 MB) | ||
| # The following list of files and their sizes is just informational / for review, and isn't used in the test: | ||
| _CodeSignature/CodeResources: 3,997 bytes (3.9 KB = 0.0 MB) | ||
| archived-expanded-entitlements.xcent: 384 bytes (0.4 KB = 0.0 MB) | ||
| Info.plist: 1,134 bytes (1.1 KB = 0.0 MB) | ||
| Microsoft.iOS.dll: 153,088 bytes (149.5 KB = 0.1 MB) | ||
| Info.plist: 1,161 bytes (1.1 KB = 0.0 MB) | ||
| Microsoft.iOS.dll: 154,112 bytes (150.5 KB = 0.1 MB) | ||
| PkgInfo: 8 bytes (0.0 KB = 0.0 MB) | ||
| runtimeconfig.bin: 1,405 bytes (1.4 KB = 0.0 MB) | ||
| SizeTestApp: 2,391,264 bytes (2,335.2 KB = 2.3 MB) | ||
| SizeTestApp: 2,391,632 bytes (2,335.6 KB = 2.3 MB) | ||
| SizeTestApp.dll: 7,680 bytes (7.5 KB = 0.0 MB) | ||
| System.Private.CoreLib.aotdata.arm64: 41,392 bytes (40.4 KB = 0.0 MB) | ||
| System.Private.CoreLib.dll: 1,003,520 bytes (980.0 KB = 1.0 MB) | ||
| System.Private.CoreLib.aotdata.arm64: 41,424 bytes (40.5 KB = 0.0 MB) | ||
| System.Private.CoreLib.dll: 1,007,104 bytes (983.5 KB = 1.0 MB) | ||
| System.Runtime.dll: 5,120 bytes (5.0 KB = 0.0 MB) | ||
| System.Runtime.InteropServices.dll: 8,192 bytes (8.0 KB = 0.0 MB) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,8 +1,8 @@ | ||
| AppBundleSize: 2,486,550 bytes (2,428.3 KB = 2.4 MB) | ||
| AppBundleSize: 2,486,702 bytes (2,428.4 KB = 2.4 MB) | ||
| # The following list of files and their sizes is just informational / for review, and isn't used in the test: | ||
| _CodeSignature/CodeResources: 2,589 bytes (2.5 KB = 0.0 MB) | ||
| archived-expanded-entitlements.xcent: 384 bytes (0.4 KB = 0.0 MB) | ||
| Info.plist: 1,169 bytes (1.1 KB = 0.0 MB) | ||
| Info.plist: 1,161 bytes (1.1 KB = 0.0 MB) | ||
| PkgInfo: 8 bytes (0.0 KB = 0.0 MB) | ||
| runtimeconfig.bin: 1,808 bytes (1.8 KB = 0.0 MB) | ||
| SizeTestApp: 2,480,592 bytes (2,422.5 KB = 2.4 MB) | ||
| SizeTestApp: 2,480,752 bytes (2,422.6 KB = 2.4 MB) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -344,11 +344,6 @@ void GenerateMainImpl (StringWriter sw, Abi abi) | |||||||||
| sw.WriteLine (); | ||||||||||
| sw.WriteLine (assembly_externs); | ||||||||||
|
|
||||||||||
| if (app.PublishReadyToRun == true) { | ||||||||||
| sw.WriteLine ("extern void* RTR_HEADER;"); | ||||||||||
| sw.WriteLine (); | ||||||||||
| } | ||||||||||
|
|
||||||||||
| sw.WriteLine ("void xamarin_register_modules_impl ()"); | ||||||||||
| sw.WriteLine ("{"); | ||||||||||
| sw.WriteLine (assembly_aot_modules); | ||||||||||
|
|
@@ -440,8 +435,6 @@ void GenerateMainImpl (StringWriter sw, Abi abi) | |||||||||
| sw.WriteLine ("\txamarin_runtime_configuration_name = {0};", string.IsNullOrEmpty (app.RuntimeConfigurationFile) ? "NULL" : $"\"{app.RuntimeConfigurationFile}\""); | ||||||||||
| if (app.Registrar == RegistrarMode.ManagedStatic) | ||||||||||
| sw.WriteLine ("\txamarin_set_is_managed_static_registrar (true);"); | ||||||||||
|
||||||||||
| sw.WriteLine ("\txamarin_set_is_managed_static_registrar (true);"); | |
| sw.WriteLine ("\txamarin_set_is_managed_static_registrar (true);"); | |
| if (app.XamarinRuntime == XamarinRuntime.NativeAOT) | |
| sw.WriteLine ("\txamarin_rtr_header = &RTR_HEADER;"); |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This task writes out C/C++ code using MSBuild item metadata without validating or escaping it. If
ModuleNameorSymbolNamemetadata is missing/empty, the generated file will contain invalid declarations/initializers, and ifModuleNamecontains characters that need escaping in a C string literal (e.g.\or"), the generated source will not compile. Consider validating required metadata for every item (and logging an MSBuild error when missing) and C-escapingmoduleNamebefore embedding it into the string literal.