RFR: 8141501: Problems with BitMap buffer management

Stefan Karlsson stefan.karlsson at oracle.com
Tue May 3 20:24:13 UTC 2016


Hi Kim,

Can I consider this patch Reviewed now?

Thanks,
StefanK

On 2016-05-03 21:53, Kim Barrett wrote:
>> On May 2, 2016, at 5:00 AM, Stefan Karlsson <stefan.karlsson at oracle.com> wrote:
>>
>> Hi Kim,
>>
>> Updated webrevs:
>> http://cr.openjdk.java.net/~stefank/8141501/webrev.03.delta
>> http://cr.openjdk.java.net/~stefank/8141501/webrev.03
>>
>> Comments inlined:
>>
>> On 2016-04-30 19:01, Kim Barrett wrote:
> […]
>>> src/share/vm/c1/c1_LinearScan.cpp
>>>   718     block->set_live_in  (ResourceBitMap(live_size)); block->live_in().clear();
>>>   719     block->set_live_out (ResourceBitMap(live_size)); block->live_out().clear();
>>>
>>> Consider instead
>>>
>>>    block->live_in().reinitialize(live_size);
>>>    block->live_out().reinitialize(live_size);
>> I intend to remove the call to clear() with the fix for this bug:
>> https://bugs.openjdk.java.net/browse/JDK-8155638
>>
>> So, I'd like to do this suggestion as a part of that change.
> OK.
>



More information about the hotspot-dev mailing list