RFR: 8139457: Relax alignment of array elements [v53]

[Roman Kennke]

On Thu, 22 Feb 2024 13:45:57 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> src/hotspot/share/oops/arrayOop.hpp line 59:
>> 
>>> 57:   // aligned 0 mod 8.  The typeArrayOop itself must be aligned at least this
>>> 58:   // strongly.
>>> 59:   static bool element_type_should_be_aligned(BasicType type) {
>> 
>> The comment above makes it sound like this is only valid for `typeArrayOop`. Should be updated to reflect the change.
>
> Really nitpicky, but why even say "of an arrayOop"? What other use of "element" within the class arrayOopDesc would make sense here? Also, it does not check (which implies asserts or similar). Why not just  
> 
> Given a type, return true if elements of that type must be aligned to 64-bit.
> 
> or similar.

Ok.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/11044#discussion_r1499484217