RFR: Small fixes - shipment one

Robbin Ehn rehn at openjdk.java.net
Tue Mar 22 15:12:58 UTC 2022


On Tue, 22 Mar 2022 14:48:34 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Reducing the number of variables, the number of assignment to each variable, white-spaces, renamed some variables, removed unnecessary template, etc...
>> Except the removal of template 'bottom', the code is the same. (assert is moved up one level)
>> 
>> Passes 1-3 (no new failures)
>> It seem to be slightly faster than before, but at least on par.
>> 
>> This is not complete at all, but to avoid merge conflict I want to ship, and also make someone happy about removing a template.
>
> src/hotspot/share/runtime/continuation.cpp line 1847:
> 
>> 1845:     assert((intptr_t)chunk->start_address() % 8 == 0, "");
>> 1846: 
>> 1847:     init_chunk(chunk);
> 
> I tried taking this out as an experiment because I thought after Java heap allocation, it would be already zeroed, but it wasn't.

Yes, the allocation returns memory without zeroing it.

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

PR: https://git.openjdk.java.net/loom/pull/114


More information about the loom-dev mailing list