-
Notifications
You must be signed in to change notification settings - Fork 554
[assembly-preparer] Create. #22392
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
base: main
Are you sure you want to change the base?
[assembly-preparer] Create. #22392
Conversation
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
d857ef1 to
71e3557
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
* Enable nullability. * Remove legacy Xamarin code. * Misc improvements / simplifications.
* Enable nullability. * Remove dead code.
* Enable nullability. * Remove dead code.
* We only support 64-bit architectures now. * We don't have any code paths conditions on IntPtr.Size, so the size benefits for the optimizations are slim to non-existent (it's used in a few mathematical expressions, pointer arithmetic, etc., nothing that affects final app size if IntPtr.Size gets inlinsed).
…nker steps. 🚧 Very early draft, no reviewing necessary 🚧 Create an 'assembly-preparer' tool which will do what the existing custom linker steps will do, and add all the scaffolding around it: * Add a new _PrepareAssembly target and a PrepareAssemblies task that calls the new tool. * Add a unit test project, which is able to run tests directly using the new tool (and not necessarily through the new MSBuild target/task, to speed up execution and make debugging *much* easier). * Add the new unit test project to xharness [PENDING]. * Make the new tool do one thing the custom linker steps currently do: preserve block code handlers. Add corresponding tests as well. * Add an opt-in .NET unit test to check the speed of using (and not using) our new assembly preparing logic. Currently the monotouch-test project is slightly faster (which doesn't make much sense, but 🤷), and the MySimpleApp is ever so slightly slower.
This reverts commit 33a2def. needs CoreTypeMap step first.
5455fca to
6d60ecc
Compare
✅ [CI Build #6d60ecc] Build passed (Build packages) ✅Pipeline on Agent |
✅ [PR Build #6d60ecc] Build passed (Detect API changes) ✅Pipeline on Agent |
✅ API diff for current PR / commitNET (empty diffs)✅ API diff vs stableNET (empty diffs)ℹ️ Generator diffGenerator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes) Pipeline on Agent |
🔥 [CI Build #6d60ecc] Build failed (Build macOS tests) 🔥Build failed for the job 'Build macOS tests' (with job status 'Failed') Pipeline on Agent |
🔥 [CI Build #6d60ecc] Test results 🔥Test results❌ Tests failed on VSTS: test results 0 tests crashed, 85 tests failed, 61 tests passed. Failures❌ dotnettests tests (iOS)1 tests failed, 0 tests passed.Failed tests
Html Report (VSDrops) Download ❌ dotnettests tests (MacCatalyst)1 tests failed, 0 tests passed.Failed tests
Html Report (VSDrops) Download ❌ dotnettests tests (macOS)1 tests failed, 0 tests passed.Failed tests
Html Report (VSDrops) Download ❌ dotnettests tests (Multiple platforms)1 tests failed, 0 tests passed.Failed tests
Html Report (VSDrops) Download ❌ dotnettests tests (tvOS)1 tests failed, 0 tests passed.Failed tests
Html Report (VSDrops) Download ❌ fsharp tests1 tests failed, 3 tests passed.Failed tests
Html Report (VSDrops) Download ❌ interdependent-binding-projects tests1 tests failed, 3 tests passed.Failed tests
Html Report (VSDrops) Download ❌ linker tests52 tests failed, 16 tests passed.Failed tests
Html Report (VSDrops) Download ❌ monotouch tests (iOS)5 tests failed, 5 tests passed.Failed tests
Html Report (VSDrops) Download ❌ monotouch tests (MacCatalyst)8 tests failed, 4 tests passed.Failed tests
Html Report (VSDrops) Download ❌ monotouch tests (macOS)4 tests failed, 9 tests passed.Failed tests
Html Report (VSDrops) Download ❌ monotouch tests (tvOS)5 tests failed, 5 tests passed.Failed tests
Html Report (VSDrops) Download ❌ windows testsDetails# 🎉 All 3 tests passed 🎉Html Report (VSDrops) Download ❌ xcframework tests3 tests failed, 1 tests passed.Failed tests
Html Report (VSDrops) Download Successes✅ cecil: All 1 tests passed. Html Report (VSDrops) Download Pipeline on Agent |
[assembly-preparer] Create a new tool to eventually replace custom linker steps.
🚧 Very early draft, no reviewing necessary 🚧
Create an 'assembly-preparer' tool which will do what the existing custom linker
steps will do, and add all the scaffolding around it:
new tool.
(and not necessarily through the new MSBuild target/task, to speed up execution
and make debugging much easier).
block code handlers. Add corresponding tests as well.
new assembly preparing logic. Currently the monotouch-test project is slightly
faster (which doesn't make much sense, but 🤷), and the MySimpleApp is ever so
slightly slower.