RFR: 8360192: C2: Make the type of count leading/trailing zero nodes more precise [v10]

Emanuel Peter epeter at openjdk.org
Tue Aug 19 14:05:47 UTC 2025


On Tue, 19 Aug 2025 13:50:28 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Qizheng Xing has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove redundant `@require` in IR test
>
> src/hotspot/share/opto/countbitsnode.cpp line 57:
> 
>> 55:   const TypeInt* ti = t->is_int();
>> 56:   return TypeInt::make(count_leading_zeros_int(~ti->_bits._zeros),
>> 57:                        count_leading_zeros_int(ti->_bits._ones),
> 
> I think this is correct, but I would like to see a short comment why it is correct.

Same in other cases below

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25928#discussion_r2285345674


More information about the hotspot-compiler-dev mailing list