> Hi all,
> 
> Vector API fails to work when:
>  - case 1: MaxVectorSize is set to <=8, or
>  - case 2: C2 is disabled
> 
> The reason is that {max/preferred} VectorShape initialization fails in both 
> cases.
> And the root cause is that VectorSupport_GetMaxLaneCount [1] returns 
> unreasonable values (0 for case 1 and -1 for case 2).
> 
> Vector API should not depend on C2 to run.
> It should work even there is no JIT compiler since it's a Java-level api.
> So let's fix it.
> 
> Testing:
>   - jdk/incubator/vector with -XX:MaxVectorSize=default/8 on Linux/x64
> 
> Thanks.
> Best regards,
> Jie
> 
> [1] 
> https://github.com/openjdk/jdk/blob/master/src/hotspot/share/prims/vectorSupport.cpp#L364

Jie Fu 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 seven additional commits since the last 
revision:

 - Fix in jdk/incubator/vector/VectorShape.java
 - Merge branch 'master' into JDK-8262096
 - Revert changes
 - Remove -XX:TieredStopAtLevel=3
 - Update the jtreg test
 - The numerator should be 8 (byte)
 - 8262096: Vector API fails to work due to VectorShape initialization exception

-------------

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/2722/files
  - new: https://git.openjdk.java.net/jdk/pull/2722/files/bbe6150c..b67b232d

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=2722&range=04
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=2722&range=03-04

  Stats: 7047 lines in 380 files changed: 3707 ins; 1675 del; 1665 mod
  Patch: https://git.openjdk.java.net/jdk/pull/2722.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/2722/head:pull/2722

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

Reply via email to