Re: RFR: 8273459: Update code segment alignment to 64 bytes [v4]

2021-09-29 Thread Jie Fu
On Tue, 28 Sep 2021 17:31:24 GMT, Scott Gibbons wrote: >> Change the default code entry alignment to 64 bytes from 32 bytes. This >> allows for maintaining proper 64-byte alignment of data within a code >> segment, which is required by several AVX-512 instructions. >> >> I ran into this whil

Re: RFR: 8273459: Update code segment alignment to 64 bytes [v4]

2021-09-28 Thread Scott Gibbons
On Tue, 28 Sep 2021 17:46:44 GMT, Vladimir Kozlov wrote: >> Scott Gibbons has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Merge branch 'asgibbons-align-fix' of https://github.com/asgibbons/jdk >> into asgibbons-align-fix >> - Revert

Re: RFR: 8273459: Update code segment alignment to 64 bytes [v4]

2021-09-28 Thread Sandhya Viswanathan
On Tue, 28 Sep 2021 17:31:24 GMT, Scott Gibbons wrote: >> Change the default code entry alignment to 64 bytes from 32 bytes. This >> allows for maintaining proper 64-byte alignment of data within a code >> segment, which is required by several AVX-512 instructions. >> >> I ran into this whil

Re: RFR: 8273459: Update code segment alignment to 64 bytes [v4]

2021-09-28 Thread Vladimir Kozlov
On Tue, 28 Sep 2021 17:31:24 GMT, Scott Gibbons wrote: >> Change the default code entry alignment to 64 bytes from 32 bytes. This >> allows for maintaining proper 64-byte alignment of data within a code >> segment, which is required by several AVX-512 instructions. >> >> I ran into this whil

Re: RFR: 8273459: Update code segment alignment to 64 bytes [v4]

2021-09-28 Thread Scott Gibbons
On Tue, 28 Sep 2021 17:31:24 GMT, Scott Gibbons wrote: >> Change the default code entry alignment to 64 bytes from 32 bytes. This >> allows for maintaining proper 64-byte alignment of data within a code >> segment, which is required by several AVX-512 instructions. >> >> I ran into this whil

Re: RFR: 8273459: Update code segment alignment to 64 bytes [v4]

2021-09-28 Thread Sandhya Viswanathan
On Tue, 28 Sep 2021 17:31:24 GMT, Scott Gibbons wrote: >> Change the default code entry alignment to 64 bytes from 32 bytes. This >> allows for maintaining proper 64-byte alignment of data within a code >> segment, which is required by several AVX-512 instructions. >> >> I ran into this whil

Re: RFR: 8273459: Update code segment alignment to 64 bytes [v4]

2021-09-28 Thread Jatin Bhateja
On Tue, 28 Sep 2021 17:31:24 GMT, Scott Gibbons wrote: >> Change the default code entry alignment to 64 bytes from 32 bytes. This >> allows for maintaining proper 64-byte alignment of data within a code >> segment, which is required by several AVX-512 instructions. >> >> I ran into this whil

Re: RFR: 8273459: Update code segment alignment to 64 bytes [v4]

2021-09-28 Thread Vladimir Kozlov
On Tue, 28 Sep 2021 17:31:24 GMT, Scott Gibbons wrote: >> Change the default code entry alignment to 64 bytes from 32 bytes. This >> allows for maintaining proper 64-byte alignment of data within a code >> segment, which is required by several AVX-512 instructions. >> >> I ran into this whil

Re: RFR: 8273459: Update code segment alignment to 64 bytes [v4]

2021-09-28 Thread Scott Gibbons
> Change the default code entry alignment to 64 bytes from 32 bytes. This > allows for maintaining proper 64-byte alignment of data within a code > segment, which is required by several AVX-512 instructions. > > I ran into this while implementing Base64 encoding and decoding. Code > segments

Re: RFR: 8273459: Update code segment alignment to 64 bytes [v3]

2021-09-28 Thread Scott Gibbons
On Tue, 28 Sep 2021 16:21:59 GMT, Scott Gibbons wrote: >> Change the default code entry alignment to 64 bytes from 32 bytes. This >> allows for maintaining proper 64-byte alignment of data within a code >> segment, which is required by several AVX-512 instructions. >> >> I ran into this whil

Re: RFR: 8273459: Update code segment alignment to 64 bytes [v3]

2021-09-28 Thread Scott Gibbons
> Change the default code entry alignment to 64 bytes from 32 bytes. This > allows for maintaining proper 64-byte alignment of data within a code > segment, which is required by several AVX-512 instructions. > > I ran into this while implementing Base64 encoding and decoding. Code > segments

Re: RFR: 8273459: Update code segment alignment to 64 bytes

2021-09-28 Thread Vladimir Kozlov
On Tue, 28 Sep 2021 01:10:53 GMT, Scott Gibbons wrote: >> Change the default code entry alignment to 64 bytes from 32 bytes. This >> allows for maintaining proper 64-byte alignment of data within a code >> segment, which is required by several AVX-512 instructions. >> >> I ran into this whil

Re: RFR: 8273459: Update code segment alignment to 64 bytes [v2]

2021-09-28 Thread Vladimir Kozlov
On Tue, 28 Sep 2021 01:15:30 GMT, Scott Gibbons wrote: >> Change the default code entry alignment to 64 bytes from 32 bytes. This >> allows for maintaining proper 64-byte alignment of data within a code >> segment, which is required by several AVX-512 instructions. >> >> I ran into this whil

Re: RFR: 8273459: Update code segment alignment to 64 bytes [v2]

2021-09-28 Thread Vladimir Kozlov
On Thu, 16 Sep 2021 16:08:22 GMT, Erik Joelsson wrote: >> Scott Gibbons has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Revert alignment of 64-bytes; Add align64() > > .gitignore line 19: > >> 17: **/JTwork/** >> 18: /src/utils/LogCompi

Re: RFR: 8273459: Update code segment alignment to 64 bytes

2021-09-28 Thread Jatin Bhateja
On Tue, 28 Sep 2021 01:10:53 GMT, Scott Gibbons wrote: >> Change the default code entry alignment to 64 bytes from 32 bytes. This >> allows for maintaining proper 64-byte alignment of data within a code >> segment, which is required by several AVX-512 instructions. >> >> I ran into this whil

Re: RFR: 8273459: Update code segment alignment to 64 bytes [v2]

2021-09-27 Thread Scott Gibbons
> Change the default code entry alignment to 64 bytes from 32 bytes. This > allows for maintaining proper 64-byte alignment of data within a code > segment, which is required by several AVX-512 instructions. > > I ran into this while implementing Base64 encoding and decoding. Code > segments

Re: RFR: 8273459: Update code segment alignment to 64 bytes

2021-09-27 Thread Scott Gibbons
On Thu, 16 Sep 2021 13:52:24 GMT, Scott Gibbons wrote: > Change the default code entry alignment to 64 bytes from 32 bytes. This > allows for maintaining proper 64-byte alignment of data within a code > segment, which is required by several AVX-512 instructions. > > I ran into this while imp

Re: RFR: 8273459: Update code segment alignment to 64 bytes

2021-09-27 Thread Vladimir Kozlov
On Thu, 16 Sep 2021 13:52:24 GMT, Scott Gibbons wrote: > Change the default code entry alignment to 64 bytes from 32 bytes. This > allows for maintaining proper 64-byte alignment of data within a code > segment, which is required by several AVX-512 instructions. > > I ran into this while imp

Re: RFR: 8273459: Update code segment alignment to 64 bytes

2021-09-27 Thread Sandhya Viswanathan
On Fri, 17 Sep 2021 14:00:44 GMT, Scott Gibbons wrote: >> Change the default code entry alignment to 64 bytes from 32 bytes. This >> allows for maintaining proper 64-byte alignment of data within a code >> segment, which is required by several AVX-512 instructions. >> >> I ran into this whil

Re: RFR: 8273459: Update code segment alignment to 64 bytes

2021-09-17 Thread Dean Long
On Thu, 16 Sep 2021 13:52:24 GMT, Scott Gibbons wrote: > Change the default code entry alignment to 64 bytes from 32 bytes. This > allows for maintaining proper 64-byte alignment of data within a code > segment, which is required by several AVX-512 instructions. > > I ran into this while imp

Re: RFR: 8273459: Update code segment alignment to 64 bytes

2021-09-17 Thread Jatin Bhateja
On Fri, 17 Sep 2021 14:00:44 GMT, Scott Gibbons wrote: > I think I have not made the point clearly enough. The `align` function is > used to manipulate the address bits for the byte following the `align()`. > This means that wherever the code is copied, the address of that byte should > have

Re: RFR: 8273459: Update code segment alignment to 64 bytes

2021-09-17 Thread Scott Gibbons
On Thu, 16 Sep 2021 13:52:24 GMT, Scott Gibbons wrote: > Change the default code entry alignment to 64 bytes from 32 bytes. This > allows for maintaining proper 64-byte alignment of data within a code > segment, which is required by several AVX-512 instructions. > > I ran into this while imp

Re: RFR: 8273459: Update code segment alignment to 64 bytes

2021-09-16 Thread Vladimir Kozlov
On Thu, 16 Sep 2021 13:52:24 GMT, Scott Gibbons wrote: > Change the default code entry alignment to 64 bytes from 32 bytes. This > allows for maintaining proper 64-byte alignment of data within a code > segment, which is required by several AVX-512 instructions. > > I ran into this while imp

Re: RFR: 8273459: Update code segment alignment to 64 bytes

2021-09-16 Thread Erik Joelsson
On Thu, 16 Sep 2021 13:52:24 GMT, Scott Gibbons wrote: > Change the default code entry alignment to 64 bytes from 32 bytes. This > allows for maintaining proper 64-byte alignment of data within a code > segment, which is required by several AVX-512 instructions. > > I ran into this while imp

Re: RFR: 8273459: Update code segment alignment to 64 bytes

2021-09-16 Thread Jatin Bhateja
On Thu, 16 Sep 2021 13:52:24 GMT, Scott Gibbons wrote: > Change the default code entry alignment to 64 bytes from 32 bytes. This > allows for maintaining proper 64-byte alignment of data within a code > segment, which is required by several AVX-512 instructions. > > I ran into this while imp

RFR: 8273459: Update code segment alignment to 64 bytes

2021-09-16 Thread Scott Gibbons
Change the default code entry alignment to 64 bytes from 32 bytes. This allows for maintaining proper 64-byte alignment of data within a code segment, which is required by several AVX-512 instructions. I ran into this while implementing Base64 encoding and decoding. Code segments which were a