Re: RFR: 8262891: Compiler implementation for Pattern Matching for switch (Preview) [v9]

2021-06-03 Thread Jan Lahoda
> This is a preview of a patch implementing JEP 406: Pattern Matching for > switch (Preview): > https://bugs.openjdk.java.net/browse/JDK-8213076 > > The current draft of the specification is here: > http://cr.openjdk.java.net/~gbierman/jep406/jep406-20210430/specs/patterns-switch-jls.html > > A

Re: RFR: 8262891: Compiler implementation for Pattern Matching for switch (Preview) [v10]

2021-06-03 Thread Jan Lahoda
> This is a preview of a patch implementing JEP 406: Pattern Matching for > switch (Preview): > https://bugs.openjdk.java.net/browse/JDK-8213076 > > The current draft of the specification is here: > http://cr.openjdk.java.net/~gbierman/jep406/jep406-20210430/specs/patterns-switch-jls.html > > A

Re: RFR: JDK-8266254: Update to use jtreg 6 [v2]

2021-06-03 Thread Erik Joelsson
On Wed, 2 Jun 2021 20:15:55 GMT, Jonathan Gibbons wrote: >> Please review the change to update to using jtreg 6. >> >> The primary change is to the jib-profiles.js file, which specifies the >> version of jtreg to use, for those systems that rely on this file. In >> addition, the `requiredVers

Re: RFR: 8267630: Start of release updates for JDK 18

2021-06-03 Thread Erik Joelsson
On Mon, 24 May 2021 22:35:04 GMT, Joe Darcy wrote: > 8267630: Start of release updates for JDK 18 Build change looks good. Please be aware of the incoming change https://bugs.openjdk.java.net/browse/JDK-8263468 which will add another version field to version-numbers.conf, which may need to be

Re: RFR: 8268129: LibraryLookup::ofDefault leaks symbols from loaded libraries [v5]

2021-06-03 Thread Maurizio Cimadamore
> This patch overhauls the library loading mechanism used by the Foreign Linker > API. We realized that, while handy, the *default* lookup abstraction > (`LibraryLookup::ofDefault`) was behaving inconsistentlt across platforms. > > This patch replaces `LibraryLookup` with a simpler `SymbolLookup

Re: RFR: 8268129: LibraryLookup::ofDefault leaks symbols from loaded libraries [v6]

2021-06-03 Thread Maurizio Cimadamore
> This patch overhauls the library loading mechanism used by the Foreign Linker > API. We realized that, while handy, the *default* lookup abstraction > (`LibraryLookup::ofDefault`) was behaving inconsistentlt across platforms. > > This patch replaces `LibraryLookup` with a simpler `SymbolLookup

Re: RFR: 8265783: Create a separate library for x86 Intel SVML assembly intrinsics [v17]

2021-06-03 Thread Sandhya Viswanathan
> This PR contains Short Vector Math Library support related changes for > [JEP-414 Vector API (Second Incubator)](https://openjdk.java.net/jeps/414), > in preparation for when targeted. > > Intel Short Vector Math Library (SVML) based intrinsics in native x86 > assembly provide optimized imple

Re: RFR: 8268129: LibraryLookup::ofDefault leaks symbols from loaded libraries [v6]

2021-06-03 Thread Erik Joelsson
On Thu, 3 Jun 2021 16:43:51 GMT, Maurizio Cimadamore wrote: >> This patch overhauls the library loading mechanism used by the Foreign >> Linker API. We realized that, while handy, the *default* lookup abstraction >> (`LibraryLookup::ofDefault`) was behaving inconsistentlt across platforms. >>

RFR: 8268142: Switch to jdk-17+24 for macosx-aarch64 at Oracle

2021-06-03 Thread Mikael Vidstedt
With promoted builds available it's a good time to switch the boot JDK etc. - Commit messages: - Merge branch 'master' into 8268142-macos-aarch64-jdk17 - 8268142: Switch to jdk-17+24 for macosx-aarch64 at Oracle Changes: https://git.openjdk.java.net/jdk/pull/4341/files Webrev: htt

Re: RFR: 8268142: Switch to jdk-17+24 for macosx-aarch64 at Oracle

2021-06-03 Thread Erik Joelsson
On Thu, 3 Jun 2021 18:24:51 GMT, Mikael Vidstedt wrote: > With promoted builds available it's a good time to switch the boot JDK etc. Marked as reviewed by erikj (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/4341

RFR: JDK-8268185: Update GitHub Actions for jtreg 6

2021-06-03 Thread Jonathan Gibbons
Please review changes to the GitHub Actions for the repo, to use jtreg 6. There are three small parts to the change: 1. The version info is updated in `make/conf/test-dependencies` 2. The new `make/build.sh` script is used to build `jtreg`, instead of the earlier `build-all.sh` script 3. The for

Re: RFR: JDK-8268185: Update GitHub Actions for jtreg 6

2021-06-03 Thread Erik Joelsson
On Thu, 3 Jun 2021 19:05:35 GMT, Jonathan Gibbons wrote: > Please review changes to the GitHub Actions for the repo, to use jtreg 6. > > There are three small parts to the change: > > 1. The version info is updated in `make/conf/test-dependencies` > 2. The new `make/build.sh` script is used to

Integrated: JDK-8268185: Update GitHub Actions for jtreg 6

2021-06-03 Thread Jonathan Gibbons
On Thu, 3 Jun 2021 19:05:35 GMT, Jonathan Gibbons wrote: > Please review changes to the GitHub Actions for the repo, to use jtreg 6. > > There are three small parts to the change: > > 1. The version info is updated in `make/conf/test-dependencies` > 2. The new `make/build.sh` script is used to

Integrated: 8265783: Create a separate library for x86 Intel SVML assembly intrinsics

2021-06-03 Thread Sandhya Viswanathan
On Thu, 22 Apr 2021 19:07:28 GMT, Sandhya Viswanathan wrote: > This PR contains Short Vector Math Library support related changes for > [JEP-414 Vector API (Second Incubator)](https://openjdk.java.net/jeps/414), > in preparation for when targeted. > > Intel Short Vector Math Library (SVML) ba

Re: RFR: 8268129: LibraryLookup::ofDefault leaks symbols from loaded libraries [v7]

2021-06-03 Thread Maurizio Cimadamore
> This patch overhauls the library loading mechanism used by the Foreign Linker > API. We realized that, while handy, the *default* lookup abstraction > (`LibraryLookup::ofDefault`) was behaving inconsistentlt across platforms. > > This patch replaces `LibraryLookup` with a simpler `SymbolLookup

Re: RFR: 8267630: Start of release updates for JDK 18 [v2]

2021-06-03 Thread Joe Darcy
> 8267630: Start of release updates for JDK 18 Joe Darcy has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 12 additional commits since the last revision:

Integrated: 8268142: Switch to jdk-17+24 for macosx-aarch64 at Oracle

2021-06-03 Thread Mikael Vidstedt
On Thu, 3 Jun 2021 18:24:51 GMT, Mikael Vidstedt wrote: > With promoted builds available it's a good time to switch the boot JDK etc. This pull request has now been integrated. Changeset: 1b4378e5 Author:Mikael Vidstedt URL: https://git.openjdk.java.net/jdk/commit/1b4378e58e601dae37

RFR: 8268214: Use system zlib and disable dtrace when building linux-aarch64 at Oracle

2021-06-03 Thread Mikael Vidstedt
Mimic what's being done on linux-x64 for consistency. - Commit messages: - 8268214: Use system zlib and disable dtrace when building linux-aarch64 at Oracle Changes: https://git.openjdk.java.net/jdk/pull/4347/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=4347&range=0

Re: RFR: 8268214: Use system zlib and disable dtrace when building linux-aarch64 at Oracle

2021-06-03 Thread David Holmes
On Thu, 3 Jun 2021 21:24:21 GMT, Mikael Vidstedt wrote: > Mimic what's being done on linux-x64 for consistency. Seems fine - and trivial. Thanks, David - Marked as reviewed by dholmes (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/4347

Integrated: 8268214: Use system zlib and disable dtrace when building linux-aarch64 at Oracle

2021-06-03 Thread Mikael Vidstedt
On Thu, 3 Jun 2021 21:24:21 GMT, Mikael Vidstedt wrote: > Mimic what's being done on linux-x64 for consistency. This pull request has now been integrated. Changeset: e2d5ff9d Author:Mikael Vidstedt URL: https://git.openjdk.java.net/jdk/commit/e2d5ff9d456dd339ccd21df2f75c4e34e5784d9a

Re: RFR: 8267630: Start of release updates for JDK 18 [v3]

2021-06-03 Thread Joe Darcy
> 8267630: Start of release updates for JDK 18 Joe Darcy has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 13 additional commits since the last revision: