harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mikhail Fursov (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HARMONY-5024) [drlvm][helper] Register for Magic Helpers
Date Fri, 16 Nov 2007 09:35:43 GMT

    [ https://issues.apache.org/jira/browse/HARMONY-5024?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12543013
] 

Mikhail Fursov commented on HARMONY-5024:
-----------------------------------------

Pavel, here are the answers

1) The location of the implementation of a helper can be changed (gc_gen, gc_cc...). But it's
signature can't be. This is the reason to encode the signature as a part of helper specification.

2) OK

3) what is (1) ?

4)  Yes

5) If you ask about JIT code these are preliminary changes. I'm going to remove all opcode/helper
specific code from helper_inliner.cpp with the next patch. With this JIRA I just turned JIT
to work with VM's registry.

6) We do support all helpers except (instanceof,checkcast and ldinterface) on x86_64. I'm
going to check why GC helpers are off.

7) You will have an assertion in JIT in debug mode if a class of a helper is not initialized


Pavel, 
I've already committed the patch, but not all of you proposals are in commit, because I read
your message later.
I'm going to commit all of your proposals with the next minor patch.




> [drlvm][helper] Register for Magic Helpers
> ------------------------------------------
>
>                 Key: HARMONY-5024
>                 URL: https://issues.apache.org/jira/browse/HARMONY-5024
>             Project: Harmony
>          Issue Type: New Feature
>          Components: DRLVM
>            Reporter: Pavel Afremov
>            Assignee: Mikhail Fursov
>         Attachments: MagicHelperRegis.patch, MagicHelperRegis.patch, MagicHelperRegis.patch,
registry.diff
>
>
> Register for Magic Helpers implementation.

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


Mime
View raw message