RFR: 8255378: [Vector API] Remove redundant vector length check after JDK-8254814 and JDK-8255210

Andrew Dinn adinn at openjdk.java.net
Wed Oct 28 10:10:17 UTC 2020


On Sun, 25 Oct 2020 08:33:26 GMT, Jie Fu <jiefu at openjdk.org> wrote:

> Hi all,
> 
> As @iwanowww pointed out [1] that there are redundant vector length checks for reductionI [2] and reductionS [3].
> It would be better to remove them.
> 
> Testing:
>  - jdk/incubator/vector on both AVX512 and AVX256 machines
> 
> Thanks.
> Best regards,
> Jie
> 
> [1] https://github.com/openjdk/jdk/pull/791#discussion_r510687005
> [2] https://github.com/openjdk/jdk/blob/master/src/hotspot/cpu/x86/x86.ad#L4429
> [3] https://github.com/openjdk/jdk/blob/master/src/hotspot/cpu/x86/x86.ad#L4625

This is arguably trivial and needs no review but here is one anyway.

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

Marked as reviewed by adinn (Reviewer).

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


More information about the hotspot-compiler-dev mailing list