On Fri, 26 Jun 2026 10:28:47 GMT, Martin Doerr <[email protected]> wrote:

>> This PR addresses some small clean-ups as suggested in review comments:
>> https://github.com/openjdk/jdk/pull/31122#discussion_r3404416714 by 
>> @theRealAph
>> https://github.com/openjdk/jdk/pull/31122#discussion_r3429286528 by 
>> @TheRealMDoerr
>> https://github.com/openjdk/jdk/pull/31122#discussion_r3429567374 by @reinrich
>> https://github.com/openjdk/jdk/pull/31122#discussion_r3438837024 by 
>> @dean-long
>> https://github.com/openjdk/jdk/pull/31122#discussion_r3438840971 by 
>> @dean-long
>> 
>> Thanks,
>> Christian
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> src/hotspot/cpu/aarch64/c1_LIRGenerator_aarch64.cpp line 1214:
> 
>> 1212:   klass2reg_with_patching(klass_reg, obj, patching_info);
>> 1213:   bool always_slow_path = is_null_free || is_flat;
>> 1214:   __ allocate_array(reg, len, tmp1, tmp2, tmp3, tmp4, T_OBJECT, 
>> klass_reg, slow_path, true /*zero_array*/, always_slow_path);
> 
> Why did you change this only for aarch64? All other platforms have it, too.

Good point, changed it on other platforms as well.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2584#discussion_r3481235713

Reply via email to