RFR(M): 8238759: Clones should always keep the base pointer
Nils Eliasson
nils.eliasson at oracle.com
Tue Mar 3 09:38:09 UTC 2020
Thank you Tobias!
// N
On 2020-03-03 09:22, Tobias Hartmann wrote:
> Hi Nils,
>
> looks good to me. Nice cleanup!
>
> Best regards,
> Tobias
>
> On 28.02.20 16:40, Nils Eliasson wrote:
>> Hi,
>>
>> This patch changes how clone uses arraycopy nodes. Normal arraycopy nodes have the base pointer as
>> src. Arraycopys used for cloning have an interior pointer as src, pointing at where the copy should
>> start. It's not ideal that the arraycopy node is used differently, and it introduces extra
>> complexity when the base pointer is needed at a later stage.
>>
>> This patch changes clones to have the base in the src field, and puts the offset to the where we
>> want to start copy, in the pos field.
>>
>> This patch also changes shenandoah code.
>>
>> https://bugs.openjdk.java.net/browse/JDK-8238759
>> http://cr.openjdk.java.net/~neliasso/8238759/webrev.01/
>>
>> Please review,
>> Nils Eliasson
More information about the hotspot-compiler-dev
mailing list