You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Mikhail Fursov (JIRA)" <ji...@apache.org> on 2007/02/27 16:53:05 UTC
[jira] Created: (HARMONY-3257) [drlvm][em64t][win] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
[drlvm][em64t][win] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
-------------------------------------------------------------------------------------------------------------------------
Key: HARMONY-3257
URL: https://issues.apache.org/jira/browse/HARMONY-3257
Project: Harmony
Issue Type: Bug
Components: DRLVM
Environment: Windows,EM64T
Reporter: Mikhail Fursov
I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
before any method is compiled VM generates and executes these stubs:
invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
These stubs are not WIN64 EM64T ready.
For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
I tried to fix the first stub but found errors in the next ones.
Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
+ If you have problems with jitrino.dll loading during startup try to do the following:
1) remove -Dvm.jitrino.cfg=release from build.bat
2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Resolved: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Gregory Shimansky (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Gregory Shimansky resolved HARMONY-3257.
----------------------------------------
Resolution: Fixed
I've committed combined_changes_in_vm_2.patch patch from this issue. Since its summary states the problem in VM I think it is time to close it now. All other changes in JIT and to other VM helpers may go to new different issues.
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Assigned To: Gregory Shimansky
> Attachments: combined_changes_in_vm.patch, combined_changes_in_vm_2.patch, ini.diff, jit2.diff, jitrino.xml, LIL_no_tailcall_and_shadow_for_codegen.patch, shadow_for_call_callnoret.patch, win64conv.diff, win_x64_helpers_1.patch, win_x64_helpers_2.patch
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Ivan Volosyuk (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12476555 ]
Ivan Volosyuk commented on HARMONY-3257:
----------------------------------------
Calling conventions are quite different on Win_x64 and Linux_x64. It may require quite a few changes to make it work on Win64.
Working on it.
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Attachments: jitrino.xml
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Pavel Pervov (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12477360 ]
Pavel Pervov commented on HARMONY-3257:
---------------------------------------
LIL_no_tailcall_and_shadow_for_codegen.patch:
1) "call" LIL instruction now provides shadow space in stack.
2) "tailcall" instruction now produces regular call on Windows x86-64 due to shadow space support.
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Attachments: combined_changes_in_vm.patch, ini.diff, jitrino.xml, LIL_no_tailcall_and_shadow_for_codegen.patch, win64conv.diff, win_x64_helpers_1.patch
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Pavel Afremov (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Pavel Afremov updated HARMONY-3257:
-----------------------------------
Attachment: win_x64_helpers_1.patch
Patch partly fixes Win_x64 helpers.
Contains changes in :
• m2n stubs
• compile_me
• invoke_managed_func.
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Attachments: ini.diff, jitrino.xml, win_x64_helpers_1.patch
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Assigned: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Gregory Shimansky (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Gregory Shimansky reassigned HARMONY-3257:
------------------------------------------
Assignee: Gregory Shimansky
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Assigned To: Gregory Shimansky
> Attachments: combined_changes_in_vm.patch, combined_changes_in_vm_2.patch, ini.diff, jitrino.xml, LIL_no_tailcall_and_shadow_for_codegen.patch, shadow_for_call_callnoret.patch, win64conv.diff, win_x64_helpers_1.patch, win_x64_helpers_2.patch
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Pavel Pervov (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Pavel Pervov updated HARMONY-3257:
----------------------------------
Attachment: shadow_for_call_callnoret.patch
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Attachments: combined_changes_in_vm.patch, ini.diff, jitrino.xml, LIL_no_tailcall_and_shadow_for_codegen.patch, shadow_for_call_callnoret.patch, win64conv.diff, win_x64_helpers_1.patch, win_x64_helpers_2.patch
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Mikhail Fursov (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12478284 ]
Mikhail Fursov commented on HARMONY-3257:
-----------------------------------------
Pavel, why did you drop some Jitrino.OPT changes from win64conv.diff ?
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Attachments: combined_changes_in_vm.patch, combined_changes_in_vm_2.patch, ini.diff, jitrino.xml, LIL_no_tailcall_and_shadow_for_codegen.patch, shadow_for_call_callnoret.patch, win64conv.diff, win_x64_helpers_1.patch, win_x64_helpers_2.patch
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Mikhail Fursov (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mikhail Fursov updated HARMONY-3257:
------------------------------------
Attachment: win64conv.diff
win64conv.diff - support for WIN64 calling convention in JIT. There is no shadow stack support in this patch.
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Attachments: combined_changes_in_vm.patch, ini.diff, jitrino.xml, win64conv.diff, win_x64_helpers_1.patch
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Mikhail Fursov (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mikhail Fursov updated HARMONY-3257:
------------------------------------
Attachment: jit2.diff
jit2.diff -> the latest and complete JIT fixes.
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Assigned To: Gregory Shimansky
> Attachments: combined_changes_in_vm.patch, combined_changes_in_vm_2.patch, ini.diff, jit2.diff, jitrino.xml, LIL_no_tailcall_and_shadow_for_codegen.patch, shadow_for_call_callnoret.patch, win64conv.diff, win_x64_helpers_1.patch, win_x64_helpers_2.patch
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Pavel Pervov (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Pavel Pervov updated HARMONY-3257:
----------------------------------
Attachment: LIL_no_tailcall_and_shadow_for_codegen.patch
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Attachments: combined_changes_in_vm.patch, ini.diff, jitrino.xml, LIL_no_tailcall_and_shadow_for_codegen.patch, win64conv.diff, win_x64_helpers_1.patch
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (HARMONY-3257) [drlvm][em64t][win] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Mikhail Fursov (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mikhail Fursov updated HARMONY-3257:
------------------------------------
Attachment: jitrino.xml
Jitrino build file if you have problems with debug version linkage.
Not intended for inclusion.
> [drlvm][em64t][win] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> -------------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Attachments: jitrino.xml
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Pavel Afremov (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Pavel Afremov updated HARMONY-3257:
-----------------------------------
Attachment: combined_changes_in_vm_2.patch
Added new combined patch for vm changes.
So on current moment the best result on Windows x86_64 was achieved after applying:
1) jitrino.xml (16 kb)
2) win64conv.diff (8 kb)
3) combined_changes_in_vm_2.patch (this patch)
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Attachments: combined_changes_in_vm.patch, combined_changes_in_vm_2.patch, ini.diff, jitrino.xml, LIL_no_tailcall_and_shadow_for_codegen.patch, shadow_for_call_callnoret.patch, win64conv.diff, win_x64_helpers_1.patch, win_x64_helpers_2.patch
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Mikhail Fursov (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12478285 ]
Mikhail Fursov commented on HARMONY-3257:
-----------------------------------------
Please do not answer. I see that combined_changes_in_vm_2.patch does not include win64conv.diff at all.
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Attachments: combined_changes_in_vm.patch, combined_changes_in_vm_2.patch, ini.diff, jitrino.xml, LIL_no_tailcall_and_shadow_for_codegen.patch, shadow_for_call_callnoret.patch, win64conv.diff, win_x64_helpers_1.patch, win_x64_helpers_2.patch
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Gregory Shimansky (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Gregory Shimansky updated HARMONY-3257:
---------------------------------------
Summary: [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready. (was: [drlvm][em64t][win] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.)
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Attachments: jitrino.xml
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Pavel Rebriy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12478295 ]
Pavel Rebriy commented on HARMONY-3257:
---------------------------------------
I think because it is a _VM_ combine changes :)))
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Attachments: combined_changes_in_vm.patch, combined_changes_in_vm_2.patch, ini.diff, jitrino.xml, LIL_no_tailcall_and_shadow_for_codegen.patch, shadow_for_call_callnoret.patch, win64conv.diff, win_x64_helpers_1.patch, win_x64_helpers_2.patch
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Closed: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Gregory Shimansky (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Gregory Shimansky closed HARMONY-3257.
--------------------------------------
No response, assuming ok.
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Assignee: Gregory Shimansky
> Attachments: combined_changes_in_vm.patch, combined_changes_in_vm_2.patch, ini.diff, jit2.diff, jitrino.xml, LIL_no_tailcall_and_shadow_for_codegen.patch, shadow_for_call_callnoret.patch, win64conv.diff, win_x64_helpers_1.patch, win_x64_helpers_2.patch
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Pavel Afremov (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12477375 ]
Pavel Afremov commented on HARMONY-3257:
----------------------------------------
I I think that LIL_no_tailcall_and_shadow_for_codegen.patch (3 kb) patch contains disputable changes:
- if (i->u.call.k!=LCK_TailCall && !c->out_sig) ERR("call not dominated by out or in2out");
+ if (
+#ifndef _WIN64
+ i->u.call.k!=LCK_TailCall &&
+#endif
Hold on to commit, please.
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Attachments: combined_changes_in_vm.patch, ini.diff, jitrino.xml, LIL_no_tailcall_and_shadow_for_codegen.patch, win64conv.diff, win_x64_helpers_1.patch
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Pavel Afremov (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12476913 ]
Pavel Afremov commented on HARMONY-3257:
----------------------------------------
win_x64_helpers_1.patch (19 kb) is not for commit. It's development version.
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Attachments: ini.diff, jitrino.xml, win_x64_helpers_1.patch
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Pavel Rebriy (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Pavel Rebriy updated HARMONY-3257:
----------------------------------
Attachment: combined_changes_in_vm.patch
This is a combined patch for VM. It includes all previous VM patches and patch for fake arguments.
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Attachments: combined_changes_in_vm.patch, ini.diff, jitrino.xml, win_x64_helpers_1.patch
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Ivan Volosyuk (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Ivan Volosyuk updated HARMONY-3257:
-----------------------------------
Attachment: ini.diff
Some changes in init_em64t.cpp attached.
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Attachments: ini.diff, jitrino.xml
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (HARMONY-3257) [drlvm][winx64] VM's
"invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs
are not EM64T ready.
Posted by "Pavel Afremov (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/HARMONY-3257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Pavel Afremov updated HARMONY-3257:
-----------------------------------
Attachment: win_x64_helpers_2.patch
Second patch to fix Win_x64 helpers.
It contains changes for lil code generator.
Not for commit.
> [drlvm][winx64] VM's "invoke_managed_func/compile_get_compile_me_generic/m2n_gen_push_m2n" stubs are not EM64T ready.
> ---------------------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-3257
> URL: https://issues.apache.org/jira/browse/HARMONY-3257
> Project: Harmony
> Issue Type: Bug
> Components: DRLVM
> Environment: Windows,EM64T
> Reporter: Mikhail Fursov
> Attachments: combined_changes_in_vm.patch, ini.diff, jitrino.xml, LIL_no_tailcall_and_shadow_for_codegen.patch, win64conv.diff, win_x64_helpers_1.patch, win_x64_helpers_2.patch
>
>
> I tried to run simple tests with HARMONY-3196 patch applied on JET and found the following problem:
> before any method is compiled VM generates and executes these stubs:
> invoke_managed_func : vm\vmcore\src\util\em64t\base\ini_em64t.cpp
> compile_get_compile_me_generic : vm\vmcore\src\util\em64t\base\compile_em64t.cpp
> m2n_gen_push_m2n: vm\port\src\lil\em64t\pim\m2n_em64t.cpp
> These stubs are not WIN64 EM64T ready.
> For example the first stub uses 6 fake regs while on windows it must use only 4. On windows stack args offset must be adjusted too.
> I tried to fix the first stub but found errors in the next ones.
> Can any VM guru who wrote Linux EM64t integration review the stubs above for WIN64 compatbility and fix them all?
> + If you have problems with jitrino.dll loading during startup try to do the following:
> 1) remove -Dvm.jitrino.cfg=release from build.bat
> 2) use my jitrino.xml (attached). This file is not indended to be included: win32 build can be broken with it.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.