RFR (XS): JDK-8058534: Remove HeapRegion::_orig_end
Joseph Provino
joseph.provino at oracle.com
Thu Sep 18 22:43:12 UTC 2014
Kim, thanks. I made that change you suggested.
Webrev: http://cr.openjdk.java.net/~jprovino/8058534/webrev.01/
joe
On 9/18/2014 6:24 PM, Kim Barrett wrote:
> On Sep 18, 2014, at 4:32 PM, Joseph Provino <joseph.provino at oracle.com> wrote:
>> Can I have reviews for the following small change that removes the
>> _orig_end member from the HeapRegion class. The information this member
>> stores can be easily recalculated on the fly, so it is just a waste of a
>> few bytes.
>>
>> CR:https://bugs.openjdk.java.net/browse/JDK-8058534
>>
>> Webrev:http://cr.openjdk.java.net/~jprovino/8058534/webrev.00/
>>
>> Testing: jprt
> In the original code, the _orig_end member that is being removed had a descriptive
> comment. That description no longer exists in the changed code. Maybe that
> comment should be moved to the orig_end() accessor?
>
> Otherwise looks good.
>
More information about the hotspot-gc-dev
mailing list