Re: RFR: 8307375: Alignment check on layouts used as sequence element is not correct [v2]

2023-05-04 Thread Maurizio Cimadamore
> This patch fixes `Utils::checkElementAlignment` to do the right thing for > _all_ layouts. > > The current implementation is broken, as it only works correctly when the > input layout is a value layout. > Since value layouts have a size that is a power of two (and size all layouts > have alig

Re: RFR: 8307375: Alignment check on layouts used as sequence element is not correct

2023-05-04 Thread Jorn Vernee
On Wed, 3 May 2023 17:44:55 GMT, Maurizio Cimadamore wrote: > This patch fixes `Utils::checkElementAlignment` to do the right thing for > _all_ layouts. > > The current implementation is broken, as it only works correctly when the > input layout is a value layout. > Since value layouts have a

RFR: 8307375: Alignment check on layouts used as sequence element is not correct

2023-05-03 Thread Maurizio Cimadamore
This patch fixes `Utils::checkElementAlignment` to do the right thing for _all_ layouts. The current implementation is broken, as it only works correctly when the input layout is a value layout. Since value layouts have a size that is a power of two (and size all layouts have alignment that is