-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
[jit] Disable partial sharing during JITting, its only been tested in full aot mode, and doesn't seem to work with JITted methods. #7161
Conversation
… full aot mode, and doesn't seem to work with JITted methods. Fixes mono#7085.
we should add a CI lane for hybrid AOT |
It would be enough to just run this make target on normal CI. |
build |
How will this impact the hybridAOT case where the IL code has been stripped and only wrappers+trampolines are JITted? |
It won't, this only affects JITting. |
Ok, I am asking because when passing |
Yes, that was the problem, it also enabled it for the JIT where it was not tested, and doesn't seem to be working. |
@monojenkins backport 2018-02 |
@monojenkins backport 2017-12 |
… full aot mode, and doesn't seem to work with JITted methods. (mono/mono#7161) * [aot] Add hybrid aot tests. * [jit] Disable partial sharing during JITting, its only been tested in full aot mode, and doesn't seem to work with JITted methods. Fixes mono/mono#7085. Commit migrated from mono/mono@8ce0775
Fixes #7085.