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