RFR: 8217869: Add count_leading_zeros utility
Tobias Hartmann
tobias.hartmann at oracle.com
Mon Jan 28 09:54:35 UTC 2019
Hi Claes,
looks good to me too.
Best regards,
Tobias
On 28.01.19 10:36, Nils Eliasson wrote:
> A very welcome improvement.
>
> Thank you Claes!
>
> Reviewed,
>
> // Nils
>
> On 2019-01-28 10:34, Claes Redestad wrote:
>> Hi,
>>
>> adding a count_leading_zeros implementation using compiler intrinsics
>> means a straightforward optimization of RegMask::find_highest_bit. For
>> platforms lacking intrinsic support a more efficient algorithm is
>> implemented.
>>
>> Adding a 64-bit version would be straightforward, but let's do that
>> when we need it.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8217869
>> Webrev: http://cr.openjdk.java.net/~redestad/8217869/open.00/
>>
>> Testing: tier1-3
>>
>> The xlc changes are untested, so it'd be much appreciated if someone
>> can run the test on that platform.
>>
>> Thanks!
>>
>> /Claes
More information about the hotspot-compiler-dev
mailing list