harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gregory Shimansky (JIRA)" <j...@apache.org>
Subject [jira] Resolved: (HARMONY-5040) [drlvm][classloader] Java field ordering must be preserved
Date Mon, 19 Nov 2007 14:47:43 GMT

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

Gregory Shimansky resolved HARMONY-5040.
----------------------------------------

    Resolution: Fixed

Patch is applied at 596313. Please check that it actually helps with performance.

> [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
>            Assignee: Gregory Shimansky
>         Attachments: fo-stable-1.patch, 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