RFR (S) 8214341 [lworld][C1] implement anewarray for value type for C1
ioi.lam at oracle.com
Wed Nov 28 16:12:02 UTC 2018
On 11/28/18 2:43 AM, Tobias Hartmann wrote:
> Hi Ioi,
> On 28.11.18 06:55, Ioi Lam wrote:
>> The patch is on top of Fred's patch for JDK-8214307 (nested flattened fields).
>> Most of the logic is shared with NewObjectArray, so I just sprinkled
>> the code with test for value-type arrays.
> This looks good to me!
>> I tested with very basic stuff (see the added test cases 36 and 37 in
> I would prefer to have these in TestArrays.java where we already have similar/same tests (I know the
> separation of tests is not perfect).
Thanks for pointing that out. TestArrays already has the tests that I
want, so I'll revert the changes in TestBasicFunctionality and push.
More information about the valhalla-dev