RFR: JDK-8202016: Use obj+offset in interpreter array access
Roman Kennke
rkennke at redhat.com
Mon May 14 17:59:42 UTC 2018
Thanks for the reviews, Andrew!
Roman
> On 14/05/18 12:34, Roman Kennke wrote:
>> Am 14.05.2018 um 13:19 schrieb Andrew Dinn:
> . . .
>>> Well, that x86 code change looks ok/ish/ -- although the need to reload
>>> from the stack is a tad disappointing.
>>>
>>> However, I am left wondering why this is not a problem on other
>>> architectures? Indeed, looking at AArch64 I believe it is actually a
>>> serious headache.
>>
>> It is, and I have a solution here:
>> cr.openjdk.java.net/~rkennke/interp_primitives_aarch64/webrev.0
>
> Yuck, how repulsive :]
>
>> I will propose this for upstream soon.
>
> Ok, so ack the original x86 change.
>
>> For AArch64 it will basically reshuffle address calculation from
>> currently (base + index) + disp (which we can't resolve) to base +
>> (index + disp) which we can.
>>
>> The alternative would be to explicitely drag the base_obj through all
>> the BarrierSetAssembler calls. So far I could live without it (x86 and
>> aarch64).
> Yes, even more repulsive, I guess.
>
> regards,
>
>
> Andrew Dinn
> -----------
> Senior Principal Software Engineer
> Red Hat UK Ltd
> Registered in England and Wales under Company Registration No. 03798903
> Directors: Michael Cunningham, Michael ("Mike") O'Neill, Eric Shander
>
More information about the hotspot-dev
mailing list