On Fri, 12 Jun 2026 15:23:54 GMT, Andrew Haley <[email protected]> wrote:

>> David Simms has updated the pull request with a new target base due to a 
>> merge or a rebase. The pull request now contains 2798 commits:
>> 
>>  - Merge remote-tracking branch 'valhalla/lworld' into 
>> jep401_sub_review_8317278
>>  - 8386239: [lworld] Update jdk/java/util/Arrays/ArraysEqCmpTest.java
>>    
>>    Reviewed-by: liach
>>  - 8386242: [lworld] Simplify and clarify StrictProcessor
>>    
>>    Reviewed-by: dsimms
>>  - 8386140: [lworld] FieldReflector using wrong Class argument
>>    
>>    Reviewed-by: alanb, dsimms
>>  - 8386216: [lworld] Rollback meaningless diff in EventClassBuilder
>>    
>>    Reviewed-by: dsimms
>>  - 8385170: [lworld] Serialization spec needs to allow abstract value 
>> classes like Number
>>    
>>    Reviewed-by: liach
>>  - 8385980: [lworld] Standardize pattern for preview value class generation
>>    
>>    Reviewed-by: liach
>>  - 8386086: [lworld] 
>> sun/jvmstat/monitor/MonitoredVm/MonitorVmStartTerminate.java should be 
>> removed from problem list
>>    
>>    Reviewed-by: liach, dcubed
>>  - Merge
>>    
>>    Merge jdk-28+1
>>  - 8386090: [lworld] Redundant test changes in lworld versus mainline
>>    
>>    Reviewed-by: vromero
>>  - ... and 2788 more: https://git.openjdk.org/jdk/compare/92298786...f76a5106
>
> src/hotspot/cpu/aarch64/c1_LIRGenerator_aarch64.cpp line 1213:
> 
>> 1211: 
>> 1212:   klass2reg_with_patching(klass_reg, obj, patching_info);
>> 1213:   __ allocate_array(reg, len, tmp1, tmp2, tmp3, tmp4, T_OBJECT, 
>> klass_reg, slow_path, true, is_null_free || is_flat);
> 
> Suggestion:
> 
>   bool always_slow_path = is_null_free || is_flat;
>   __ allocate_array(reg, len, tmp1, tmp2, tmp3, tmp4, T_OBJECT, klass_reg, 
> slow_path, /*zero_array*/true, always_slow_path);

Addressed with [JDK-8387274](https://bugs.openjdk.org/browse/JDK-8387274).

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31122#discussion_r3472432246

Reply via email to