Re: RFR: JDK-8278014: [vectorapi] Remove test run script [v2]

2021-12-01 Thread Aleksey Shipilev 
On Tue, 30 Nov 2021 23:35:08 GMT, Paul Sandoz  wrote:

>> Remove Vector API scripts for building and running tests. `jtreg` should be 
>> used instead.
>> 
>> Also updated the test generation script to remove options that assume 
>> mercurial as the code repository.
>
> Paul Sandoz has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Update copywrite year.

Marked as reviewed by shade (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/6621

Re: RFR: JDK-8278014: [vectorapi] Remove test run script [v2]

2021-11-30 Thread Paul Sandoz 
> Remove Vector API scripts for building and running tests. `jtreg` should be 
> used instead.
> 
> Also updated the test generation script to remove options that assume 
> mercurial as the code repository.

Paul Sandoz has updated the pull request incrementally with one additional 
commit since the last revision:

  Update copywrite year.

-

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/6621/files
  - new: https://git.openjdk.java.net/jdk/pull/6621/files/65fa1fbe..4a5bf3b9

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=6621&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=6621&range=00-01

  Stats: 3 lines in 3 files changed: 0 ins; 0 del; 3 mod
  Patch: https://git.openjdk.java.net/jdk/pull/6621.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/6621/head:pull/6621

PR: https://git.openjdk.java.net/jdk/pull/6621

Re: RFR: JDK-8278014: [vectorapi] Remove test run script [v2]

2021-11-30 Thread Jie Fu 
On Tue, 30 Nov 2021 23:31:21 GMT, Paul Sandoz  wrote:

>> Remove Vector API scripts for building and running tests. `jtreg` should be 
>> used instead.
>> 
>> Also updated the test generation script to remove options that assume 
>> mercurial as the code repository.
>
> Paul Sandoz has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Update copywrite year.

Thanks for your update.
LGTM

-

Marked as reviewed by jiefu (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/6621