[vectorIntrinsics] RFR: 8283598: [vectorapi] Add new vector operation for compress bits
Xiaohong Gong
xgong at openjdk.java.net
Tue Mar 29 09:48:56 UTC 2022
On Thu, 24 Mar 2022 06:31:31 GMT, Smita Kamath <svkamath at openjdk.org> wrote:
> Hi,
>
> I've added support for new vector operations for compressing bits of integral vector types(Byte/Short/Integer/Long).
> The implementation is based on Compress or Generalized Extract mentioned in Hackers Delight by Henry S. Warren, Jr.
> The implementation does the following: given a mask and the number to be compressed, the bits of the number corresponding to the set mask bit are selected and compressed.
>
> Currently, this PR addresses only Java changes for compress bits operation. I've also updated the test framework.
> Do review and share feedback.
src/hotspot/share/prims/vectorSupport.cpp line 538:
> 536: case T_INT:
> 537: case T_LONG: return Op_CompressBits;
> 538: default: fatal("COMPRESS_BITS: %s", type2name(bt));
The identity issue "three spaces before `"case"`". Seems the above bitcount and reverse have the same issue. So please fix them together. Thanks!
-------------
PR: https://git.openjdk.java.net/panama-vector/pull/184
More information about the panama-dev
mailing list