RFR: 8287738: [PPC64] jdk/incubator/vector/*VectorTests failing [v2]

Goetz Lindenmaier goetz at openjdk.java.net
Thu Jun 9 09:53:24 UTC 2022


On Thu, 9 Jun 2022 09:48:33 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

>> `PopCountVI` needs to handle several types after [JDK-8284960](https://bugs.openjdk.java.net/browse/JDK-8284960).
>> (See https://github.com/openjdk/jdk/commit/6f6486e97743eadfb20b4175e1b4b2b05b59a17a.)
>
> Martin Doerr has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Support T_LONG instead of T_DOUBLE. Neither of these types are currently used.

LGTM

src/hotspot/cpu/ppc/ppc.ad line 14117:

> 14115:         __ vpopcntw($dst$$VectorSRegister->to_vr(), $src$$VectorSRegister->to_vr());
> 14116:         break;
> 14117:       case T_DOUBLE:

I think this should be T_LONG.

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

Marked as reviewed by goetz (Reviewer).

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


More information about the hotspot-compiler-dev mailing list