Bug 57476 - AOT fails with large StructLayout size on Debug+ARMv7
Summary: AOT fails with large StructLayout size on Debug+ARMv7
Status: CONFIRMED
Alias: None
Product: iOS
Classification: Xamarin
Component: Mono runtime / AOT compiler (show other bugs)
Version: XI 10.10 (d15-2)
Hardware: Macintosh Mac OS
: Normal normal
Target Milestone: Future Cycle (TBD)
Assignee: Zoltan Varga
URL:
Depends on:
Blocks:
 
Reported: 2017-06-14 10:01 UTC by Tom Gilder
Modified: 2017-06-22 18:21 UTC (History)
5 users (show)

See Also:
Tags:
Is this bug a regression?: No
Last known good build:


Attachments
Example repro solution (7.90 KB, application/zip)
2017-06-14 10:02 UTC, Tom Gilder
Details
Full build log with -v -v -v (28.67 KB, text/plain)
2017-06-14 10:03 UTC, Tom Gilder
Details
Environment info (1.88 KB, text/plain)
2017-06-14 10:04 UTC, Tom Gilder
Details

Description Tom Gilder 2017-06-14 10:01:31 UTC
# Steps to reproduce

1. Create a struct with a size above 4100 - [StructLayout(LayoutKind.Auto, Size = 4101)]   
2. Ensure iOS debugging is enabled (MtouchDebug=true)
2. Attempt to compile for a 32-bit iOS Device on ARMv7

# Expected behavior

Compiles successfully

# Actual behavior

Fails with "Could not AOT the assembly"

# Supplemental info (logs, images, videos)

Logs and repro projet to come
Comment 1 Tom Gilder 2017-06-14 10:02:34 UTC
Created attachment 22878 [details]
Example repro solution

Repro solution - ensure you're building for Debug
Comment 2 Tom Gilder 2017-06-14 10:03:41 UTC
Created attachment 22879 [details]
Full build log with -v -v -v
Comment 3 Tom Gilder 2017-06-14 10:04:44 UTC
Created attachment 22880 [details]
Environment info
Comment 4 Vincent Dondain [MSFT] 2017-06-14 22:19:07 UTC
I managed to successfully reproduce this issue using the above repro steps and the following environment: https://gist.github.com/VincentDondain/a84e7b2cc058b8b890c98752809a57e0

This bug doesn't seem to be a regression as it does not work with (XI 10.8 / Mono 4.8) either: https://gist.github.com/VincentDondain/d99af3a7917c25825d3d52ce1a1e419f.
Comment 5 Zoltan Varga 2017-06-20 18:37:24 UTC
https://github.com/mono/mono/pull/5078

Note that this only affects 32 bit.
Comment 6 Rolf Bjarne Kvinge [MSFT] 2017-06-21 10:45:25 UTC
@Zoltan, can this be back ported to 2017-06?
Comment 7 Zoltan Varga 2017-06-21 18:30:46 UTC
The arm builds failed on wrench, so the fix might not be good.
Comment 8 Zoltan Varga 2017-06-22 18:21:34 UTC
https://github.com/mono/mono/pull/5094

Notice (2018-05-21): bugzilla.xamarin.com will be switching to read-only mode on Thursday, 2018-05-25 22:00 UTC.

Please join us on Visual Studio Developer Community and GitHub to continue tracking issues. Bugzilla will remain available for reference in read-only mode. We will continue to work on open Bugzilla bugs and copy them to the new locations as needed for follow-up. The See Also field on each Bugzilla bug will be updated with a link to its new location when applicable.

After Bugzilla is read-only, if you have new information to add for a bug that does not yet have a matching issue on Developer Community or GitHub, you can create a follow-up issue in the new location. Copy and paste the title and description from this bug, and then add your new details. You can get a pre-formatted version of the title and description here:

In special cases you might also want the comments:

Our sincere thanks to everyone who has contributed on this bug tracker over the years. Thanks also for your understanding as we make these adjustments and improvements for the future.

Note You need to log in before you can comment on or make changes to this bug.