RFR: 8264324: Simplify allocation list management in OopStorage::reduce_deferred_updates [v2]

Tue Mar 30 08:38:13 UTC 2021

On Tue, 30 Mar 2021 08:34:23 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

>> Please review this cleanup of the allocation list management performed by OopStorage::reduce_deferred_updates.  Part of its job is to take a block whose allocation set has changed and update its place in the allocation list accordingly.  The old code has somewhat confusing logic and does unnecessary work in some cases.  The new code is clearer and avoids that excess work.  (I'm not sure what I was thinking when I wrote the old code.)
>> 
>> Testing:
>> mach5 tier1
>> I've also been using this patch for some time under other in-development changes that make heavy use of OopStorage.
>
> Kim Barrett has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Improve comment per ayang discussion

Marked as reviewed by ayang (Committer).

-------------

PR: https://git.openjdk.java.net/jdk/pull/3227