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

[WASM] Mixed Mode: function signature mismatch #13463

Closed
jeromelaban opened this issue Mar 14, 2019 · 1 comment · Fixed by #13548
Assignees
Labels

Comments

@jeromelaban
Copy link
Contributor

@jeromelaban jeromelaban commented Mar 14, 2019

Using the sample here: https://github.com/jeromelaban/Wasm.Samples/tree/master/MixedMode.Tests

The built website is using mixed mode compilation, with some assemblies explicitly excluded. I could not get down to a smaller repro.

Steps to Reproduce

  1. Using linux (and the setup outlined here), build the application using
    msbuild /r /p:Configuration=Release
  2. Run the application in FF or Chrome in the bin\Release\netstandard2.0\dist using the server.py

Current Behavior

uno-bootstrap.js:147 RuntimeError: function signature mismatch
    at _Uno_Foundation_Uno_Extensions_NotifyCollectionChangedEventArgsExtensions_ToNotifyCollectionChangedEventArgs_Windows_Foundation_Collections_IVectorChangedEventArgs (wasm-function[7855]:320)
    at _aot_wrapper_gsharedvt_out_sig_u4_u4 (wasm-function[8333]:18)
    at _jit_call_cb (wasm-function[16434]:143)
    at dynCall_vi (wasm-function[17338]:5)
    at Module.dynCall_vi (http://localhost:8000/mono.js:13810:38)
    at invoke_vi (http://localhost:8000/mono.js:10324:5)
    at _mono_llvm_cpp_catch_exception (wasm-function[10866]:20)
    at _do_jit_call (wasm-function[16420]:957)
    at _interp_exec_method_full (wasm-function[16410]:25137)
    at _interp_exec_method_full (wasm-function[16410]:22575)

Expected Behavior

OnItemsSourceSingleCollectionChanged is displayed in the debugging console.

On which platforms did you notice this

Version Used: 8fed666

@lewing lewing added the target-wasm label Mar 14, 2019
@vargaz

This comment has been minimized.

Copy link
Member

@vargaz vargaz commented Mar 19, 2019

Have some difficulty building this. Could you attach the obj and bin directories for the app ?

vargaz added a commit to vargaz/mono that referenced this issue Mar 20, 2019
vargaz added a commit to vargaz/mono that referenced this issue Mar 20, 2019
vargaz added a commit to vargaz/mono that referenced this issue Mar 21, 2019
monojenkins added a commit that referenced this issue Mar 21, 2019
…nterpreter. (#13548)

[llvmonly] Add gsharedvt in wrappers to calls which could go to the interpreter.

Fixes #13463.




<!--
Thank you for your Pull Request!

If you are new to contributing to Mono, please try to do your best at conforming to our coding guidelines http://www.mono-project.com/community/contributing/coding-guidelines/ but don't worry if you get something wrong. One of the project members will help you to get things landed.

Does your pull request fix any of the existing issues? Please use the following format: Fixes #issue-number
-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.