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

Vladimir Ivanov vlivanov at openjdk.java.net
Mon Oct 26 17:00:18 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

Looks good.

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

Marked as reviewed by vlivanov (Reviewer).

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


More information about the hotspot-compiler-dev mailing list