-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
AOT compiler for .NET/iOS not built with MONOTOUCH defined #45002
Comments
I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label. |
/cc @vargaz |
This should be fixed; does this particular crash still happen? (I haven't personally gotten to the point where I've run an AOT-compiled app on-device, so it might take me a little while to create and test a repro.) |
I'll check once there are .NET packages with the fix. |
This is fixed. |
Description
The symptom is that an AOT-compiled iOS app can crash at runtime on device with:
I diagnosed a bit, and the reason is that this line is never executed:
runtime/src/mono/mono/mini/aot-compiler.c
Lines 13778 to 13780 in d85f060
which presumably happens because
MONOTOUCH
isn't defined.I decompiled the AOT compiler shipped with .NET, and the one we currently use for Xamarin.iOS, and only the latter contains the line above.
.Mono for Xamarin.iOS: https://gist.github.com/rolfbjarne/8607804c44f873ebf6ef711e50f30134#file-foo-diff-L217-L223
.NET: https://gist.github.com/rolfbjarne/37cebf4f43296d9f51474944dd424650#file-foo-diff-L219-L224
This is the missing line: https://gist.github.com/rolfbjarne/8607804c44f873ebf6ef711e50f30134#file-foo-diff-L222
CC @akoeplinger
The text was updated successfully, but these errors were encountered: