Re: RFR: 8286473: Drop --enable-preview from Record related tests [v2]

2022-05-11 Thread Aleksey Shipilev
On Tue, 10 May 2022 14:54:39 GMT, Aleksey Shipilev wrote: >> There are plenty of tests failing on many architectures due to >> `--enable-preview` VM code introduced by Loom. This improvements eliminates >> some of the redundant `--enable-preview` clauses from the Record tests, >> since

Re: RFR: 8286473: Drop --enable-preview from Record related tests [v2]

2022-05-10 Thread Mandy Chung
On Tue, 10 May 2022 14:54:39 GMT, Aleksey Shipilev wrote: >> There are plenty of tests failing on many architectures due to >> `--enable-preview` VM code introduced by Loom. This improvements eliminates >> some of the redundant `--enable-preview` clauses from the Record tests, >> since

Re: RFR: 8286473: Drop --enable-preview from Record related tests [v2]

2022-05-10 Thread Jaikiran Pai
On Tue, 10 May 2022 14:54:39 GMT, Aleksey Shipilev wrote: >> There are plenty of tests failing on many architectures due to >> `--enable-preview` VM code introduced by Loom. This improvements eliminates >> some of the redundant `--enable-preview` clauses from the Record tests, >> since

Re: RFR: 8286473: Drop --enable-preview from Record related tests [v2]

2022-05-10 Thread Alan Bateman
On Tue, 10 May 2022 14:51:52 GMT, Aleksey Shipilev wrote: >> There are plenty of tests failing on many architectures due to >> `--enable-preview` VM code introduced by Loom. This improvements eliminates >> some of the redundant `--enable-preview` clauses from the Record tests, >> since

Re: RFR: 8286473: Drop --enable-preview from Record related tests [v2]

2022-05-10 Thread Aleksey Shipilev
On Tue, 10 May 2022 14:43:06 GMT, Alan Bateman wrote: >> Aleksey Shipilev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Review comments > > test/jdk/java/nio/Buffer/BulkPutBuffer.java line 57: > >> 55: * @run testng/othervm

Re: RFR: 8286473: Drop --enable-preview from Record related tests [v2]

2022-05-10 Thread Aleksey Shipilev
> There are plenty of tests failing on many architectures due to > `--enable-preview` VM code introduced by Loom. This improvements eliminates > some of the redundant `--enable-preview` clauses from the Record tests, since > Records have been graduated from preview in JDK 16. > > Additional