dragonwell-project / dragonwell21

GNU General Public License v2.0
66 stars 19 forks source link

[Backport] 8139457: Relax alignment of array elements #67

Closed linade closed 3 months ago

linade commented 3 months ago

This pull request backports the following patches:

8139457: Relax alignment of array elements 8327426: RISC-V: Move alignment shim into initialize_header() in C1_MacroAssembler::allocate_array 8331098: [Aarch64] Fix crash in Arrays.equals() intrinsic with -CCP

All applies cleanly. Additionally we fix one typo.

8139457 and 8331098 changes alignment requirement for array headers, and fix one bug that's caused. 8327426 is only there to make the code look more like the lilliput21 branch, so that future lilliput patches apply more easily.

CLAassistant commented 3 months ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 3 committers have signed the CLA.

:x: rkennke
:x: zifeihan
:x: linade
You have signed the CLA already but the status is still pending? Let us recheck it.

mmyxym commented 3 months ago

As the commits are clean backports, could you please describe the additional typo change?

linade commented 3 months ago

As the commits are clean backports, could you please describe the additional typo change?

doing a diff with lilliput21 branch reveals this typo

weixlu commented 3 months ago

LGTM