[janino-dev] [jira] Created: (JANINO-130) wide versions of IINC and RET are broken

View: New views
3 Messages — Rating Filter:   Alert me  

[janino-dev] [jira] Created: (JANINO-130) wide versions of IINC and RET are broken

by JIRA jira@codehaus.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

wide versions of IINC and RET are broken
----------------------------------------

                 Key: JANINO-130
                 URL: http://jira.codehaus.org/browse/JANINO-130
             Project: Janino
          Issue Type: Bug
            Reporter: Matt Fowles
            Assignee: Arno Unkrig
         Attachments: wide.diff

The JVM requires wide instructions for IINC or RET opcodes when there are more than 255 local variables.  The attached patch implements that (including tests).

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email



[janino-dev] [jira] Commented: (JANINO-130) wide versions of IINC and RET are broken

by JIRA jira@codehaus.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message


    [ http://jira.codehaus.org/browse/JANINO-130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=163351#action_163351 ]

Matt Fowles commented on JANINO-130:
------------------------------------

The patch also reworks the initialization of

OpCode.WIDE_OPCODE_PROPERTIES

because somewhere the long list got out of sync, and it was far easier to ensure the correct offsets get hit programmatically instead of statically.

> wide versions of IINC and RET are broken
> ----------------------------------------
>
>                 Key: JANINO-130
>                 URL: http://jira.codehaus.org/browse/JANINO-130
>             Project: Janino
>          Issue Type: Bug
>            Reporter: Matt Fowles
>            Assignee: Arno Unkrig
>         Attachments: wide.diff
>
>
> The JVM requires wide instructions for IINC or RET opcodes when there are more than 255 local variables.  The attached patch implements that (including tests).

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email



[jira] Commented: (JANINO-130) wide versions of IINC and RET are broken

by JIRA jira@codehaus.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message


    [ http://jira.codehaus.org/browse/JANINO-130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=179475#action_179475 ]

Matt Fowles commented on JANINO-130:
------------------------------------

You can find a fixed version of this at http://code.google.com/p/janino-streambase/

> wide versions of IINC and RET are broken
> ----------------------------------------
>
>                 Key: JANINO-130
>                 URL: http://jira.codehaus.org/browse/JANINO-130
>             Project: Janino
>          Issue Type: Bug
>            Reporter: Matt Fowles
>            Assignee: Arno Unkrig
>         Attachments: wide.diff
>
>
> The JVM requires wide instructions for IINC or RET opcodes when there are more than 255 local variables.  The attached patch implements that (including tests).

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email