Integrated: 8284992: Fix misleading Vector API doc for LSHR operator
Jie Fu
jiefu at openjdk.java.net
Fri Apr 29 23:05:44 UTC 2022
On Tue, 19 Apr 2022 08:41:50 GMT, Jie Fu <jiefu at openjdk.org> wrote:
> Hi all,
>
> The Current Vector API doc for `LSHR` is
>
> Produce a>>>(n&(ESIZE*8-1)). Integral only.
>
>
> This is misleading which may lead to bugs for Java developers.
> This is because for negative byte/short elements, the results computed by `LSHR` will be different from that of `>>>`.
> For more details, please see https://github.com/openjdk/jdk/pull/8276#issue-1206391831 .
>
> After the patch, the doc for `LSHR` is
>
> Produce zero-extended right shift of a by (n&(ESIZE*8-1)) bits. Integral only.
>
>
> Thanks.
> Best regards,
> Jie
This pull request has now been integrated.
Changeset: e54f26aa
Author: Jie Fu <jiefu at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/e54f26aa3d5d44264e052bc51d3d819a8da5d1e7
Stats: 4 lines in 1 file changed: 2 ins; 0 del; 2 mod
8284992: Fix misleading Vector API doc for LSHR operator
Reviewed-by: psandoz
-------------
PR: https://git.openjdk.java.net/jdk/pull/8291
More information about the core-libs-dev
mailing list