harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ilya Berezhniuk (JIRA)" <j...@apache.org>
Subject [jira] Updated: (HARMONY-5065) [drlvm][jit] OpcodeDesc structure is padded incorrectly
Date Wed, 07 Nov 2007 14:49:50 GMT

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

Ilya Berezhniuk updated HARMONY-5065:

    Attachment: H-5065.patch

the same patch without trailing workspaces.

> [drlvm][jit] OpcodeDesc structure is padded incorrectly
> -------------------------------------------------------
>                 Key: HARMONY-5065
>                 URL: https://issues.apache.org/jira/browse/HARMONY-5065
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>            Reporter: Ilya Berezhniuk
>            Priority: Minor
>         Attachments: H-5065.patch, H-5065.patch, H-5065.patch
> OpcodeDesc structure is padded to 128 bytes with char[] array which size is calculated
using a sum of other fields size.
> This approach does not consider default enum size and field alignment.
> I've checked sizeof(OpcodeDesc) on IA-32 Windows/Linux, it's 132 bytes. So if the goal
was performance gain, then this goal is not achieved.
> Probably #pragma pack(1) should be used to eliminate field alignment, and sizeof() should
be used for padding size calculation.
> On the other hand, unaligned fields can also affect performance, so additional padding
should be used for frequently accessed fields.

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

View raw message