Skip to content
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

Fails to compile without AOT (enable-minimal=profiler,debug,aot) #8430

Closed
hmargies opened this issue Apr 24, 2018 · 6 comments · Fixed by xamarin/xamarin-android#1981

Comments

@hmargies
Copy link

@hmargies hmargies commented Apr 24, 2018

Mono fails to compile when AOT is disabled (enable-minimal=profiler,debug,aot).

Steps to Reproduce

sh autogen.sh
./configure --enable-minimal=profiler,debug,aot
make

Current Behavior

The function mono_aot_get_weak_field_indexes is used in mini-runtime.c and implemented in aot-runtime.c but not compiled in due to DISABLE_AOT being defined.

  CC       mono_boehm-main.o
  CCLD     mono-boehm
./.libs/libmini.a(libmini_la-mini-runtime.o): In function `mini_init':
/home/henry/Development/test/mono/mono/mini/mini-runtime.c:4041: undefined reference to `mono_aot_get_weak_field_indexes'
collect2: error: ld returned 1 exit status
make[4]: *** [mono-boehm] Error 1
make[4]: Leaving directory `/home/henry/Development/test/mono/mono/mini'
make[3]: *** [all] Error 2
make[3]: Leaving directory `/home/henry/Development/test/mono/mono/mini'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/home/henry/Development/test/mono/mono'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/home/henry/Development/test/mono'
make: *** [all] Error 2

On which platforms did you notice this

[ ] macOS
[X ] Linux
[ ] Windows

Version Used: 5.10.1.47 and also master from GitHub

@mikaelj

This comment has been minimized.

Copy link

@mikaelj mikaelj commented May 30, 2018

Any progress?

@marek-safar

This comment has been minimized.

Copy link
Member

@marek-safar marek-safar commented May 30, 2018

/cc @vargaz

vargaz added a commit to vargaz/mono that referenced this issue May 30, 2018
@vargaz vargaz closed this in #8952 May 30, 2018
vargaz added a commit that referenced this issue May 30, 2018
Fixes #8430.
baulig added a commit to baulig/mono that referenced this issue May 31, 2018
commit da2de44
Author: Martin Baulig <mabaul@microsoft.com>
Date:   Thu May 31 10:46:00 2018 -0400

    [corlib]: Add `System.HashCode` - revisiting mono#8822 and mono#8879. (mono#8934)

    Enable `System.HashCode` again, cleanup the `.sources` files and fix the Windows build.

commit e5cd5a9
Author: Martin Baulig <mabaul@microsoft.com>
Date:   Wed May 30 18:49:20 2018 -0400

    [build]: Make 'gensources' look for platform-specific .sources files as well.

commit 23fd728
Author: Nick Craver <nrcraver@gmail.com>
Date:   Wed May 30 21:29:17 2018 -0400

    Issue template: update help link to https://

    Since GitHub pages rolled out HTTPS support, links to `www.mono-project.com` can now be `https://` :) This also removes a `www.` 301 users currently hits.

commit c972668
Author: Zoltan Varga <vargaz@gmail.com>
Date:   Wed May 30 19:52:50 2018 -0400

    [runtime] Fix the --enable-minimal=aot build. (mono#8952)

    Fixes mono#8430.

commit 3a630bf
Author: monojenkins <jo.shields+jenkins@xamarin.com>
Date:   Wed May 30 21:59:09 2018 +0000

    Bump API snapshot submodule

commit 8afcf1b
Author: EgorBo <egorbo@gmail.com>
Date:   Wed May 30 21:05:12 2018 +0300

    remove `implicit operator String (ReadOnlySpan<char> value)`
baulig added a commit to baulig/mono that referenced this issue May 31, 2018
commit da2de44
Author: Martin Baulig <mabaul@microsoft.com>
Date:   Thu May 31 10:46:00 2018 -0400

    [corlib]: Add `System.HashCode` - revisiting mono#8822 and mono#8879. (mono#8934)

    Enable `System.HashCode` again, cleanup the `.sources` files and fix the Windows build.

commit e5cd5a9
Author: Martin Baulig <mabaul@microsoft.com>
Date:   Wed May 30 18:49:20 2018 -0400

    [build]: Make 'gensources' look for platform-specific .sources files as well.

commit 23fd728
Author: Nick Craver <nrcraver@gmail.com>
Date:   Wed May 30 21:29:17 2018 -0400

    Issue template: update help link to https://

    Since GitHub pages rolled out HTTPS support, links to `www.mono-project.com` can now be `https://` :) This also removes a `www.` 301 users currently hits.

commit c972668
Author: Zoltan Varga <vargaz@gmail.com>
Date:   Wed May 30 19:52:50 2018 -0400

    [runtime] Fix the --enable-minimal=aot build. (mono#8952)

    Fixes mono#8430.

commit 3a630bf
Author: monojenkins <jo.shields+jenkins@xamarin.com>
Date:   Wed May 30 21:59:09 2018 +0000

    Bump API snapshot submodule

commit 8afcf1b
Author: EgorBo <egorbo@gmail.com>
Date:   Wed May 30 21:05:12 2018 +0300

    remove `implicit operator String (ReadOnlySpan<char> value)`
baulig added a commit to baulig/mono that referenced this issue May 31, 2018
[corefx] Import System.Hashcode (mono#8822)

* [System.Native] Add pal_random

* Bring HashCode.cs.

* Use platform-specific .sources files.

* Bump API snapshot submodule

Fix the .sources mess.

Bump corefx.

Move `HashCode` into `corlib.dll.sources` and make most of the profiles platform-specific.

Squashed commit of the following:

commit da2de44
Author: Martin Baulig <mabaul@microsoft.com>
Date:   Thu May 31 10:46:00 2018 -0400

    [corlib]: Add `System.HashCode` - revisiting mono#8822 and mono#8879. (mono#8934)

    Enable `System.HashCode` again, cleanup the `.sources` files and fix the Windows build.

commit e5cd5a9
Author: Martin Baulig <mabaul@microsoft.com>
Date:   Wed May 30 18:49:20 2018 -0400

    [build]: Make 'gensources' look for platform-specific .sources files as well.

commit 23fd728
Author: Nick Craver <nrcraver@gmail.com>
Date:   Wed May 30 21:29:17 2018 -0400

    Issue template: update help link to https://

    Since GitHub pages rolled out HTTPS support, links to `www.mono-project.com` can now be `https://` :) This also removes a `www.` 301 users currently hits.

commit c972668
Author: Zoltan Varga <vargaz@gmail.com>
Date:   Wed May 30 19:52:50 2018 -0400

    [runtime] Fix the --enable-minimal=aot build. (mono#8952)

    Fixes mono#8430.

commit 3a630bf
Author: monojenkins <jo.shields+jenkins@xamarin.com>
Date:   Wed May 30 21:59:09 2018 +0000

    Bump API snapshot submodule

commit 8afcf1b
Author: EgorBo <egorbo@gmail.com>
Date:   Wed May 30 21:05:12 2018 +0300

    remove `implicit operator String (ReadOnlySpan<char> value)`

Bump api-snapshot
baulig added a commit to baulig/mono that referenced this issue May 31, 2018
[corefx] Import System.Hashcode (mono#8822)

* [System.Native] Add pal_random

* Bring HashCode.cs.

* Use platform-specific .sources files.

* Bump API snapshot submodule

Fix the .sources mess.

Bump corefx.

Move `HashCode` into `corlib.dll.sources` and make most of the profiles platform-specific.

Squashed commit of the following:

commit da2de44
Author: Martin Baulig <mabaul@microsoft.com>
Date:   Thu May 31 10:46:00 2018 -0400

    [corlib]: Add `System.HashCode` - revisiting mono#8822 and mono#8879. (mono#8934)

    Enable `System.HashCode` again, cleanup the `.sources` files and fix the Windows build.

commit e5cd5a9
Author: Martin Baulig <mabaul@microsoft.com>
Date:   Wed May 30 18:49:20 2018 -0400

    [build]: Make 'gensources' look for platform-specific .sources files as well.

commit 23fd728
Author: Nick Craver <nrcraver@gmail.com>
Date:   Wed May 30 21:29:17 2018 -0400

    Issue template: update help link to https://

    Since GitHub pages rolled out HTTPS support, links to `www.mono-project.com` can now be `https://` :) This also removes a `www.` 301 users currently hits.

commit c972668
Author: Zoltan Varga <vargaz@gmail.com>
Date:   Wed May 30 19:52:50 2018 -0400

    [runtime] Fix the --enable-minimal=aot build. (mono#8952)

    Fixes mono#8430.

commit 3a630bf
Author: monojenkins <jo.shields+jenkins@xamarin.com>
Date:   Wed May 30 21:59:09 2018 +0000

    Bump API snapshot submodule

commit 8afcf1b
Author: EgorBo <egorbo@gmail.com>
Date:   Wed May 30 21:05:12 2018 +0300

    remove `implicit operator String (ReadOnlySpan<char> value)`

Bump api-snapshot
baulig added a commit to baulig/mono that referenced this issue May 31, 2018
commit da2de44
Author: Martin Baulig <mabaul@microsoft.com>
Date:   Thu May 31 10:46:00 2018 -0400

    [corlib]: Add `System.HashCode` - revisiting mono#8822 and mono#8879. (mono#8934)

    Enable `System.HashCode` again, cleanup the `.sources` files and fix the Windows build.

commit e5cd5a9
Author: Martin Baulig <mabaul@microsoft.com>
Date:   Wed May 30 18:49:20 2018 -0400

    [build]: Make 'gensources' look for platform-specific .sources files as well.

commit 23fd728
Author: Nick Craver <nrcraver@gmail.com>
Date:   Wed May 30 21:29:17 2018 -0400

    Issue template: update help link to https://

    Since GitHub pages rolled out HTTPS support, links to `www.mono-project.com` can now be `https://` :) This also removes a `www.` 301 users currently hits.

commit c972668
Author: Zoltan Varga <vargaz@gmail.com>
Date:   Wed May 30 19:52:50 2018 -0400

    [runtime] Fix the --enable-minimal=aot build. (mono#8952)

    Fixes mono#8430.

commit 3a630bf
Author: monojenkins <jo.shields+jenkins@xamarin.com>
Date:   Wed May 30 21:59:09 2018 +0000

    Bump API snapshot submodule

commit 8afcf1b
Author: EgorBo <egorbo@gmail.com>
Date:   Wed May 30 21:05:12 2018 +0300

    remove `implicit operator String (ReadOnlySpan<char> value)`

commit 42b9868
Merge: 7912154 bc4a1b3
Author: Alexis Christoforides <alexis@thenull.net>
Date:   Wed May 30 15:31:49 2018 -0400

    Merge pull request mono#8719 from mono/fix-gh-8712

    [System.IO] Finish initializing all FileSystemWatchers, not just the f…

commit 7912154
Author: Marek Safar <marek.safar@gmail.com>
Date:   Tue May 29 13:57:45 2018 +0200

    [acceptance-tests] Bump roslyn tests to match bundled version

commit bc4a1b3
Author: Alexis Christoforides <alexis@thenull.net>
Date:   Wed May 30 09:54:02 2018 -0400

    [System.IO] Dispatch events asynchronously in FileSystemWatcher.DispatchEvents()

    This used to happen in RaiseEvents(), but some CoreFX FSW tests require that a derived FSW be able to fire its event synchronously.

commit 2fe6c6b
Author: Alexis Christoforides <alexis@thenull.net>
Date:   Thu May 17 20:09:41 2018 -0400

    [System.IO] Prevent the CoreFX bridge from even try-entering a monitor.

    In the case of the FSEvent FileSystemWatcher, the CFRunLoop event loop thread has already an interrupt handler installed in the icall, and mono_monitor_try_enter  may try to install one as well which is not supported. So we create separate event tasks before we use the Operation() method.

commit 2731ac9
Author: Bernhard Urban <bernhard.urban@xamarin.com>
Date:   Wed May 30 15:40:09 2018 +0200

    [native crash] limit output of process mapping (mono#8910)

    [native crash] limit output of process mapping

    less noise

commit a7f8569
Author: Bernhard Urban <bernhard.urban@xamarin.com>
Date:   Wed May 30 15:25:08 2018 +0200

    [native crash] dump memory around instruction pointer (mono#8911)

    [native crash] dump memory around instruction pointer

    It can help to identify issues in generated code. We don't want to ship a disassembler, so we just do a hexdump. It looks like this:

    ```
    Memory around native instruction pointer (0x10ca1b50e):
    0x10ca1b4fe  48 c7 04 24 00 00 00 00 48 8b 44 24 10 48 8b f8  H..$....H.D$.H..
    0x10ca1b50e  83 38 00 48 8d 64 24 00 49 bb 3a b5 a1 0c 01 00  .8.H.d$.I.:.....
    0x10ca1b51e  00 00 41 ff d3 48 89 04 24 e9 00 00 00 00 48 8b  ..A..H..$.....H.
    0x10ca1b52e  04 24 e9 00 00 00 00 48 83 c4 18 c3 e8 c1 1c 53  .$.....H.......S
    ```

    Offline tools can be used to disassemble the code.

commit 3a8218e
Author: Alexander Köplinger <alex.koeplinger@outlook.com>
Date:   Wed May 30 02:01:07 2018 +0200

    [bcl] Fix ONO_PATH warning during resgen (mono#8933)

    We'd get the following warning during the build:

    ```
    '/Users/alexander/dev/mono/mcs/class/System.Data.Services.Client/ONO_PATH' in MONO_PATH doesn't exist or has wrong permissions.
    ```

    The reason is that we didn't escape the $ symbol in the resgen template
    so the embedded $$MONO_PATH in the RESGEN variable ended up being mangled.

    I'm kinda surprised this worked at all for so many years ¯\_(ツ)_/¯

commit f37db9c
Author: Marek Safar <marek.safar@gmail.com>
Date:   Wed May 30 00:31:51 2018 +0200

    [System.Web] Fix bug with not preserving state of CheckBoxList after postback because of value beeing rendered (mono#8928)

commit 6a7bd7b
Author: Marek Safar <marek.safar@gmail.com>
Date:   Wed May 30 00:22:36 2018 +0200

    [Facades] Adds few netcore extensions (mono#8907)

    * [Facades] Adds few netcore extensions

    Fixes mono#7947

    * Bump API snapshot submodule

    * [csproj] Update project files

commit 4adf710
Author: Ankit Jain <radical@gmail.com>
Date:   Tue May 29 18:10:37 2018 -0400

    Bump binary-reference-assemblies (mono#8929)

    Includes the fix for https://bugzilla.xamarin.com/show_bug.cgi?id=23668
    via mono/reference-assemblies#7

commit dd6fe29
Author: Ankit Jain <radical@gmail.com>
Date:   Tue May 29 18:09:31 2018 -0400

    [roslyn][msbuild] Copy the new Microsoft.Managed.Core.targets file in roslyn. (mono#8931)

    Fixes mono#8897 .

commit 183d994
Author: Alexis Christoforides <alexis@thenull.net>
Date:   Tue May 15 07:50:39 2018 -0400

    [System.IO] Finish initializing all FileSystemWatchers, not just the first one

    Fixes mono#8712
baulig added a commit to baulig/mono that referenced this issue May 31, 2018
commit 349fd61
Author: Martin Baulig <mabaul@microsoft.com>
Date:   Thu May 31 16:04:16 2018 -0400

    [corefx] Enable HashCode on all profiles.

    * HashCode.cs has been moved into `corlib.dll.sources`.

    * Add `PLATFORMS = darwin linux win32` to the following profiles:
      basic, orbis, testing_aot_full, testing_aot_hybrid, unreal, wasm.

    * Add per-platform sources which are now used by all profiles which
      do not have an explicit profile-platform sources file:
      - `darwin_corlib.dll.sources`
      - `linux_corlib.dll.sources`
      - `win32_corlib.dll.sources`

    * Add new `monodroid_corlib.dll.sources`.

    * Add new `xammac_net_4_5_corlib.dll.sources`.

    * Remove `testing_aot_full_corlib_test.dll.sources`.

    * Add the platform-specific non-cryptographic RNG to all profiles.

commit da2de44
Author: Martin Baulig <mabaul@microsoft.com>
Date:   Thu May 31 10:46:00 2018 -0400

    [corlib]: Add `System.HashCode` - revisiting mono#8822 and mono#8879. (mono#8934)

    Enable `System.HashCode` again, cleanup the `.sources` files and fix the Windows build.

commit e5cd5a9
Author: Martin Baulig <mabaul@microsoft.com>
Date:   Wed May 30 18:49:20 2018 -0400

    [build]: Make 'gensources' look for platform-specific .sources files as well.

commit 23fd728
Author: Nick Craver <nrcraver@gmail.com>
Date:   Wed May 30 21:29:17 2018 -0400

    Issue template: update help link to https://

    Since GitHub pages rolled out HTTPS support, links to `www.mono-project.com` can now be `https://` :) This also removes a `www.` 301 users currently hits.

commit c972668
Author: Zoltan Varga <vargaz@gmail.com>
Date:   Wed May 30 19:52:50 2018 -0400

    [runtime] Fix the --enable-minimal=aot build. (mono#8952)

    Fixes mono#8430.

commit 3a630bf
Author: monojenkins <jo.shields+jenkins@xamarin.com>
Date:   Wed May 30 21:59:09 2018 +0000

    Bump API snapshot submodule

commit 8afcf1b
Author: EgorBo <egorbo@gmail.com>
Date:   Wed May 30 21:05:12 2018 +0300

    remove `implicit operator String (ReadOnlySpan<char> value)`

commit 42b9868
Merge: 7912154 bc4a1b3
Author: Alexis Christoforides <alexis@thenull.net>
Date:   Wed May 30 15:31:49 2018 -0400

    Merge pull request mono#8719 from mono/fix-gh-8712

    [System.IO] Finish initializing all FileSystemWatchers, not just the f…

commit 7912154
Author: Marek Safar <marek.safar@gmail.com>
Date:   Tue May 29 13:57:45 2018 +0200

    [acceptance-tests] Bump roslyn tests to match bundled version

commit bc4a1b3
Author: Alexis Christoforides <alexis@thenull.net>
Date:   Wed May 30 09:54:02 2018 -0400

    [System.IO] Dispatch events asynchronously in FileSystemWatcher.DispatchEvents()

    This used to happen in RaiseEvents(), but some CoreFX FSW tests require that a derived FSW be able to fire its event synchronously.

commit 2fe6c6b
Author: Alexis Christoforides <alexis@thenull.net>
Date:   Thu May 17 20:09:41 2018 -0400

    [System.IO] Prevent the CoreFX bridge from even try-entering a monitor.

    In the case of the FSEvent FileSystemWatcher, the CFRunLoop event loop thread has already an interrupt handler installed in the icall, and mono_monitor_try_enter  may try to install one as well which is not supported. So we create separate event tasks before we use the Operation() method.

commit 2731ac9
Author: Bernhard Urban <bernhard.urban@xamarin.com>
Date:   Wed May 30 15:40:09 2018 +0200

    [native crash] limit output of process mapping (mono#8910)

    [native crash] limit output of process mapping

    less noise

commit a7f8569
Author: Bernhard Urban <bernhard.urban@xamarin.com>
Date:   Wed May 30 15:25:08 2018 +0200

    [native crash] dump memory around instruction pointer (mono#8911)

    [native crash] dump memory around instruction pointer

    It can help to identify issues in generated code. We don't want to ship a disassembler, so we just do a hexdump. It looks like this:

    ```
    Memory around native instruction pointer (0x10ca1b50e):
    0x10ca1b4fe  48 c7 04 24 00 00 00 00 48 8b 44 24 10 48 8b f8  H..$....H.D$.H..
    0x10ca1b50e  83 38 00 48 8d 64 24 00 49 bb 3a b5 a1 0c 01 00  .8.H.d$.I.:.....
    0x10ca1b51e  00 00 41 ff d3 48 89 04 24 e9 00 00 00 00 48 8b  ..A..H..$.....H.
    0x10ca1b52e  04 24 e9 00 00 00 00 48 83 c4 18 c3 e8 c1 1c 53  .$.....H.......S
    ```

    Offline tools can be used to disassemble the code.

commit 3a8218e
Author: Alexander Köplinger <alex.koeplinger@outlook.com>
Date:   Wed May 30 02:01:07 2018 +0200

    [bcl] Fix ONO_PATH warning during resgen (mono#8933)

    We'd get the following warning during the build:

    ```
    '/Users/alexander/dev/mono/mcs/class/System.Data.Services.Client/ONO_PATH' in MONO_PATH doesn't exist or has wrong permissions.
    ```

    The reason is that we didn't escape the $ symbol in the resgen template
    so the embedded $$MONO_PATH in the RESGEN variable ended up being mangled.

    I'm kinda surprised this worked at all for so many years ¯\_(ツ)_/¯

commit f37db9c
Author: Marek Safar <marek.safar@gmail.com>
Date:   Wed May 30 00:31:51 2018 +0200

    [System.Web] Fix bug with not preserving state of CheckBoxList after postback because of value beeing rendered (mono#8928)

commit 6a7bd7b
Author: Marek Safar <marek.safar@gmail.com>
Date:   Wed May 30 00:22:36 2018 +0200

    [Facades] Adds few netcore extensions (mono#8907)

    * [Facades] Adds few netcore extensions

    Fixes mono#7947

    * Bump API snapshot submodule

    * [csproj] Update project files

commit 4adf710
Author: Ankit Jain <radical@gmail.com>
Date:   Tue May 29 18:10:37 2018 -0400

    Bump binary-reference-assemblies (mono#8929)

    Includes the fix for https://bugzilla.xamarin.com/show_bug.cgi?id=23668
    via mono/reference-assemblies#7

commit dd6fe29
Author: Ankit Jain <radical@gmail.com>
Date:   Tue May 29 18:09:31 2018 -0400

    [roslyn][msbuild] Copy the new Microsoft.Managed.Core.targets file in roslyn. (mono#8931)

    Fixes mono#8897 .

commit 183d994
Author: Alexis Christoforides <alexis@thenull.net>
Date:   Tue May 15 07:50:39 2018 -0400

    [System.IO] Finish initializing all FileSystemWatchers, not just the first one

    Fixes mono#8712
@drgroot

This comment has been minimized.

Copy link

@drgroot drgroot commented Jul 1, 2018

Seem to have the same problem with version 5.12.0.226 on Linux.
Reopen?
Source: Tarball

monojenkins added a commit to monojenkins/mono that referenced this issue Jul 2, 2018
monojenkins added a commit to monojenkins/mono that referenced this issue Jul 2, 2018
@lewurm

This comment has been minimized.

Copy link
Member

@lewurm lewurm commented Jul 2, 2018

it wasn't backported to the release branches.

@marek-safar @luhenry is it okay to backport it to 2018-02 still? #9394

akoeplinger added a commit that referenced this issue Jul 2, 2018
@marek-safar marek-safar added this to the 2018-04 (5.14.xx) milestone Jul 10, 2018
jonpryor added a commit to jonpryor/xamarin-android that referenced this issue Jul 19, 2018
Fixes: mono/mono#8430
Fixes: mono/mono#9318
Fixes: xamarin#1951

Context: mono/mono#9298

Installs `cmake` as part of Android SDK provisioning.

Fix building of Windows cross-compilers on Ubuntu Linux.

Interp fixes.

Make mono-state dumper abandon stuck threads

merp-related changes.

Debugger fixes.
jonpryor added a commit to xamarin/xamarin-android that referenced this issue Jul 19, 2018
Fixes: mono/mono#8430
Fixes: mono/mono#9318
Fixes: #1951

Context: mono/mono#9298

Installs `cmake` as part of Android SDK provisioning.

Fix building of Windows cross-compilers on Ubuntu Linux.

Interp fixes.

Make mono-state dumper abandon stuck threads

merp-related changes.

Debugger fixes.
jonpryor added a commit to xamarin/xamarin-android that referenced this issue Aug 8, 2018
Fixes: #1130
Fixes: #1561 (comment)
Fixes: #1845
Fixes: #1951

Context: https://bugzilla.xamarin.com/show_bug.cgi?id=10087
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=11771
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=12850
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=18941
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=19436
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=25444
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=33208
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=58413
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=59184
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=59400
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=59779
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=60065
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=60843
Context: mono/mono#6174
Context: mono/mono#6178
Context: mono/mono#6180
Context: mono/mono#6181
Context: mono/mono#6186
Context: mono/mono#6187
Context: mono/mono#6211
Context: mono/mono#6266
Context: mono/mono#6579
Context: mono/mono#6666
Context: mono/mono#6752
Context: mono/mono#6801
Context: mono/mono#6812
Context: mono/mono#6848
Context: mono/mono#6940
Context: mono/mono#6948
Context: mono/mono#6998
Context: mono/mono#6999
Context: mono/mono#7016
Context: mono/mono#7085
Context: mono/mono#7086
Context: mono/mono#7095
Context: mono/mono#7134
Context: mono/mono#7137
Context: mono/mono#7145
Context: mono/mono#7184
Context: mono/mono#7240
Context: mono/mono#7262
Context: mono/mono#7289
Context: mono/mono#7338
Context: mono/mono#7356
Context: mono/mono#7364
Context: mono/mono#7378
Context: mono/mono#7389
Context: mono/mono#7449
Context: mono/mono#7460
Context: mono/mono#7535
Context: mono/mono#7536
Context: mono/mono#7537
Context: mono/mono#7565
Context: mono/mono#7588
Context: mono/mono#7596
Context: mono/mono#7610
Context: mono/mono#7613
Context: mono/mono#7620
Context: mono/mono#7624
Context: mono/mono#7637
Context: mono/mono#7655
Context: mono/mono#7657
Context: mono/mono#7661
Context: mono/mono#7685
Context: mono/mono#7696
Context: mono/mono#7729
Context: mono/mono#7786
Context: mono/mono#7792
Context: mono/mono#7805
Context: mono/mono#7822
Context: mono/mono#7828
Context: mono/mono#7860
Context: mono/mono#7864
Context: mono/mono#7903
Context: mono/mono#7920
Context: mono/mono#8089
Context: mono/mono#8143
Context: mono/mono#8267
Context: mono/mono#8311
Context: mono/mono#8340
Context: mono/mono#8409
Context: mono/mono#8417
Context: mono/mono#8430
Context: mono/mono#8698
Context: mono/mono#8701
Context: mono/mono#8712
Context: mono/mono#8721
Context: mono/mono#8726
Context: mono/mono#8866
Context: mono/mono#9023
Context: mono/mono#9031
Context: mono/mono#9033
Context: mono/mono#9044
Context: mono/mono#9179
Context: mono/mono#9318
Context: mono/mono#9318
Context: xamarin/maccore#628
Context: xamarin/maccore#629
Context: xamarin/maccore#673
jonpryor added a commit to xamarin/xamarin-android that referenced this issue Aug 13, 2018
Fixes: #1130
Fixes: #1561 (comment)
Fixes: #1845
Fixes: #1951

Context: https://bugzilla.xamarin.com/show_bug.cgi?id=10087
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=11771
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=12850
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=18941
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=19436
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=25444
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=33208
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=58413
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=59184
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=59400
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=59779
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=60065
Context: https://bugzilla.xamarin.com/show_bug.cgi?id=60843
Context: mono/mono#6174
Context: mono/mono#6178
Context: mono/mono#6180
Context: mono/mono#6181
Context: mono/mono#6186
Context: mono/mono#6187
Context: mono/mono#6211
Context: mono/mono#6266
Context: mono/mono#6579
Context: mono/mono#6666
Context: mono/mono#6752
Context: mono/mono#6801
Context: mono/mono#6812
Context: mono/mono#6848
Context: mono/mono#6940
Context: mono/mono#6948
Context: mono/mono#6998
Context: mono/mono#6999
Context: mono/mono#7016
Context: mono/mono#7085
Context: mono/mono#7086
Context: mono/mono#7095
Context: mono/mono#7134
Context: mono/mono#7137
Context: mono/mono#7145
Context: mono/mono#7184
Context: mono/mono#7240
Context: mono/mono#7262
Context: mono/mono#7289
Context: mono/mono#7338
Context: mono/mono#7356
Context: mono/mono#7364
Context: mono/mono#7378
Context: mono/mono#7389
Context: mono/mono#7449
Context: mono/mono#7460
Context: mono/mono#7535
Context: mono/mono#7536
Context: mono/mono#7537
Context: mono/mono#7565
Context: mono/mono#7588
Context: mono/mono#7596
Context: mono/mono#7610
Context: mono/mono#7613
Context: mono/mono#7620
Context: mono/mono#7624
Context: mono/mono#7637
Context: mono/mono#7655
Context: mono/mono#7657
Context: mono/mono#7661
Context: mono/mono#7685
Context: mono/mono#7696
Context: mono/mono#7729
Context: mono/mono#7786
Context: mono/mono#7792
Context: mono/mono#7805
Context: mono/mono#7822
Context: mono/mono#7828
Context: mono/mono#7860
Context: mono/mono#7864
Context: mono/mono#7903
Context: mono/mono#7920
Context: mono/mono#8089
Context: mono/mono#8143
Context: mono/mono#8267
Context: mono/mono#8311
Context: mono/mono#8340
Context: mono/mono#8409
Context: mono/mono#8417
Context: mono/mono#8430
Context: mono/mono#8698
Context: mono/mono#8701
Context: mono/mono#8712
Context: mono/mono#8721
Context: mono/mono#8726
Context: mono/mono#8866
Context: mono/mono#9023
Context: mono/mono#9031
Context: mono/mono#9033
Context: mono/mono#9044
Context: mono/mono#9179
Context: mono/mono#9318
Context: mono/mono#9318
Context: xamarin/maccore#628
Context: xamarin/maccore#629
Context: xamarin/maccore#673
jonpryor added a commit to xamarin/xamarin-android that referenced this issue Oct 9, 2018
Bumps to mono/llvm:release_60@117a508c
Bumps to xamarin/xamarin-android-api-compatibility:master@7ccb4802

	$ git diff --shortstat e1af6ea..ab3c897d       # mono
        1443 files changed, 66049 insertions(+), 45745 deletions(-)
	$ git diff --shortstat bdb3a116..117a508c      # llvm
	 26794 files changed, 4110589 insertions(+), 754376 deletions(-)
	$ git diff --shortstat c550d1bd..7ccb4802      # xamarin-android-api-compatibility
	 2 files changed, 16260 insertions(+), 12347 deletions(-)

Incomplete summary of easily `grep`able fixes:

Fixes: https://bugzilla.xamarin.com/show_bug.cgi?id=11199
Fixes: https://bugzilla.xamarin.com/show_bug.cgi?id=19436
Fixes: https://bugzilla.xamarin.com/show_bug.cgi?id=23668
Fixes: https://bugzilla.xamarin.com/show_bug.cgi?id=26983
Fixes: https://bugzilla.xamarin.com/show_bug.cgi?id=33728
Fixes: https://bugzilla.xamarin.com/show_bug.cgi?id=46917
fixes: https://bugzilla.xamarin.com/show_bug.cgi?id=60065
Fixes: mono/mono#6173
Fixes: mono/mono#6466
Fixes: mono/mono#6647
Fixes: mono/mono#6834
Fixes: mono/mono#7058
Fixes: mono/mono#7137
Fixes: mono/mono#7260
Fixes: mono/mono#7305
Fixes: mono/mono#7402
Fixes: mono/mono#7525
Fixes: mono/mono#7610
Fixes: mono/mono#7649
Fixes: mono/mono#7655
Fixes: mono/mono#7683
Fixes: mono/mono#7685
Fixes: mono/mono#7716
Fixes: mono/mono#7731
Fixes: mono/mono#7785
Fixes: mono/mono#7828
Fixes: mono/mono#7944
Fixes: mono/mono#7947
Fixes: mono/mono#8036
Fixes: mono/mono#8074
Fixes: mono/mono#8089
Fixes: mono/mono#8112
Fixes: mono/mono#8122
Fixes: mono/mono#8143
Fixes: mono/mono#8149
Fixes: mono/mono#8152
Fixes: mono/mono#8175
Fixes: mono/mono#8177
Fixes: mono/mono#8250
Fixes: mono/mono#8267
Fixes: mono/mono#8273
Fixes: mono/mono#8282
Fixes: mono/mono#8310
Fixes: mono/mono#8311
Fixes: mono/mono#8329
Fixes: mono/mono#8340
Fixes: mono/mono#8372
Fixes: mono/mono#8407
Fixes: mono/mono#8409
Fixes: mono/mono#8422
Fixes: mono/mono#8430
Fixes: mono/mono#8439
fixes: mono/mono#8447
Fixes: mono/mono#8469
Fixes: mono/mono#8504
Fixes: mono/mono#8575
Fixes: mono/mono#8597
Fixes: mono/mono#8623
Fixes: mono/mono#8627
Fixes: mono/mono#8698
Fixes: mono/mono#8701
Fixes: mono/mono#8712
Fixes: mono/mono#8721
Fixes: mono/mono#8726
Fixes: mono/mono#8759
Fixes: mono/mono#8787
Fixes: mono/mono#8820
Fixes: mono/mono#8848
Fixes: mono/mono#8866
Fixes: mono/mono#8897
Fixes: mono/mono#8915
Fixes: mono/mono#8970
Fixes: mono/mono#8979
Fixes: mono/mono#9023
Fixes: mono/mono#9031
Fixes: mono/mono#9033
Fixes: mono/mono#9179
Fixes: mono/mono#9234
Fixes: mono/mono#9262
Fixes: mono/mono#9277
Fixes: mono/mono#9318
Fixes: mono/mono#9542
Fixes: mono/mono#9753
Fixes: mono/mono#9839
Fixes: mono/mono#9869
Fixes: mono/mono#9870
Fixes: mono/mono#9943
Fixes: mono/mono#9996
Fixes: mono/mono#10000
Fixes: mono/mono#10303
Fixes: mono/mono#10447
Fixes: mono/mono#10483
Fixes: mono/mono#10488
Fixes: xamarin/maccore#628
Fixes: xamarin/maccore#673
Fixes: #1561 (comment)
Fixes: #1845
Fixes: xamarin/xamarin-macios#4347
Fixes: xamarin/xamarin-macios#4617
Fixes: xamarin/xamarin-macios#4618
@jbatt33

This comment has been minimized.

Copy link

@jbatt33 jbatt33 commented Nov 8, 2018

Not sure if this is fixed in 5.16 release against ARM builds. x86 appears to build fine with AOT disabled, but receiving the following error from the ARM build:

"
mono_aot_get_ftnptr_arg_trampoline
"


/libmonogc.a/stubborn.o .libs/libmonoboehm-2.0.lax/libmonogc.a/typd_mlc.o .libs/libmonoboehm-2.0.lax/libmonogc.a/win32_threads.o
| /usr/src/debug/mono/5.16.0.179-r0/mono-5.16.0.179/mono/mini/mini-trampolines.c:163: error: undefined reference to 'mono_aot_get_ftnptr_arg_trampoline'
| collect2: error: ld returned 1 exit status
| arm-angstrom-linux-gnueabi-ar: `u' modifier ignored since `D' is the default (see `U')
| Makefile:1398: recipe for target 'mono-boehm' failed
| make[4]: *** [mono-boehm] Error 1
| make[4]: *** Waiting for unfinished jobs....
| arm-angstrom-linux-gnueabi-libtool: link: (cd .libs/libmonosgen-2.0.lax/libeglib.a && arm-angstrom-linux-gnueabi-ar x "/opt/yocto/angstrom/build/tmp-angstrom-glibc/work/armv7at2hf-neon-angstrom-linux-gnueabi/mono/5.16.0.179-r0/mono-5.16.0.179/mono/mini/../../mono/eglib/.libs/libeglib.a")
| /usr/src/debug/mono/5.16.0.179-r0/mono-5.16.0.179/mono/mini/mini-trampolines.c:163: error: undefined reference to 'mono_aot_get_ftnptr_arg_trampoline'
| collect2: error: ld returned 1 exit status
| Makefile:1402: recipe for target 'mono-sgen' failed
| make[4]: *** [mono-sgen] Error 1
| arm-angstrom-linux-gnueabi-libtool: link: arm-angstrom-linux-gnueabi-ar cru .libs/libmonosgen-2.0.a   .libs/libmonosgen-2.0.lax/libmini.a/libmini_la-abcremoval.o .libs/libmonosgen-2.0.lax/libmini.a/libmini_la-alias-analysis.o .libs/libmonosgen-2.0.lax/libmini.a/libmini_la-aot-compiler.o .libs/libmonosgen-2.0.lax/libmini.a/libmini_la-aot-

What other information can I provide?

@lewurm

This comment has been minimized.

Copy link
Member

@lewurm lewurm commented Nov 12, 2018

@jbatt33 thanks for your report. Submitted a fix here: #11653

monojenkins added a commit that referenced this issue Nov 13, 2018
[arm] fix --enable-minimal=aot build

mentioned here: #8430 (comment)
EgorBo added a commit to EgorBo/mono that referenced this issue Nov 14, 2018
[arm] fix --enable-minimal=aot build

mentioned here: mono#8430 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
7 participants
You can’t perform that action at this time.