[vectorIntrinsics] RFR: 8283709: Add x86 back-end implementation for bit BIT_COUNT operation [v9]

Jatin Bhateja jbhateja at openjdk.java.net
Tue Apr 12 00:29:12 UTC 2022


On Mon, 11 Apr 2022 21:20:26 GMT, Sandhya Viswanathan <sviswanathan at openjdk.org> wrote:

>> Jatin Bhateja has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8283709: Review comments resolved.
>
> src/hotspot/cpu/x86/vm_version_x86.cpp line 1705:
> 
>> 1703: 
>> 1704:   // Use population count instruction if available.
>> 1705:   if (supports_popcnt() || supports_avx512_vpopcntdq() || supports_avx512_bitalg()) {
> 
> The UsePopCountInstruction is only for scalar popcont. Extending it to Vector popcount is causing lot of confustion. Let us keep it for scalar only as below:
> if (supports_popcnt()) {
>  ...
> }

UsePopcountInstruction implies generating direct popcount instruction for targets which support it. A bit count operation can also be implemented without popcount as depicted by this patch.

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

PR: https://git.openjdk.java.net/panama-vector/pull/185


More information about the panama-dev mailing list