harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Aleksey Shipilev (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HARMONY-5040) [drlvm][classloader] Java field ordering must be preserved
Date Fri, 09 Nov 2007 15:20:51 GMT

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

Aleksey Shipilev commented on HARMONY-5040:
-------------------------------------------

Pavel, the patch is looking good.
I vote to commit it in this form.
Thanks!

> [drlvm][classloader] Java field ordering must be preserved
> ----------------------------------------------------------
>
>                 Key: HARMONY-5040
>                 URL: https://issues.apache.org/jira/browse/HARMONY-5040
>             Project: Harmony
>          Issue Type: Improvement
>          Components: DRLVM
>            Reporter: Aleksey Shipilev
>         Attachments: fo-stable-1.patch, fo-stable-1.patch, fo-stable-1.patch
>
>
> Current implementation of class support in Harmony sorts the class fields decrementally
by size to avoid misalignment. However, it's done with quicksort which is known to be unstable
(e.g. there are no guarantees that order of elements with equal keys will not be changed).
That lead to shaking Java field order which could be optimized to achieve the spatial locality.

> Proposed patch changes the sorting algorithm to bubblesort, which is known to be stable.
That's performance-safe since average field count is not high.

-- 
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