[lworld] RFR: lworld aaload/aastore

[David Simms]

So I updated the aaload/store interpreter for lworld, adding more tests 
and covariant array copy, here's an updated webrev:

http://cr.openjdk.java.net/~dsimms/valhalla/lworld_value_aaloadstore/webrev1/

The ProfileInterpreter bug is still present however...going to need to 
understand why (doesn't appear to be stack change, could be wrong)...


On 14/03/18 15:41, David Simms wrote:
>
> Thanks for the heads up...I will double check with my latest fixes + 
> latest repo.
>
> Since posting the RFR yesterday I've increased tests and found a few 
> bugs. Also started implementing covariance and arraycopy to handle it...
>
> We'll see if I can update the webrev today or early tomorrow.
>
> /D
>
> On 14/03/18 14:21, Tobias Hartmann wrote:
>> Update: All tests pass with -XX:-ProfileInterpreter -Xint.
>>
>>
>> On 14.03.2018 14:17, Tobias Hartmann wrote:
>>> Hi David,
>>>
>>> I've executed 
>>> test/hotspot/jtreg/compiler/valhalla/valuetypes/TestArrays.java with 
>>> -Xint and your
>>> patch but I'm seeing crashes: