[jira] Closed: (HARMONY-5714) [drlvm] Remove extra synchronization runtime helpers.

["Chunrong Lai (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/HARMONY-5714?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Chunrong Lai closed HARMONY-5714.
---------------------------------

    Resolution: Fixed


  Because Buqi think that JIT should never remove the ckecknull before the vm_rt_moniter_enter, it should be safe for the stub to remove the CHECK_NULL instructions inside. So I'd like to close the issue again.
  Thanks.

> [drlvm] Remove extra synchronization runtime helpers.
> -----------------------------------------------------
>
>                 Key: HARMONY-5714
>                 URL: https://issues.apache.org/jira/browse/HARMONY-5714
>             Project: Harmony
>          Issue Type: Improvement
>          Components: DRLVM
>            Reporter: Pavel Afremov
>            Assignee: Chunrong Lai
>         Attachments: extra_strong_assert_fix.patch, extra_strong_assert_fix.patch, remove_extra_monitors_helper_1.patch, remove_extra_monitors_helper_2.patch, remove_extra_monitors_helper_3.patch
>
>
> Remove extra synchronization runtime helpers.
> All list of synchronization runtime helpers:
> •	VM_RT_MONITOR_ENTER_NON_NULL
> •	VM_RT_MONITOR_EXIT_NON_NULL
> •	VM_RT_MONITOR_ENTER_STATIC
> •	VM_RT_MONITOR_EXIT_STATIC
> •	VM_RT_MONITOR_ENTER
> •	VM_RT_MONITOR_EXIT
>  can be replace by two helpers only:
> •	VM_RT_MONITOR_ENTER
> •	VM_RT_MONITOR_EXIT
> It reduces quantity of synchronization helpers in 3 times.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.