RFR: 8315066: Add unsigned bounds and known bits to TypeInt/Long [v2]

Emanuel Peter epeter at openjdk.org
Tue Sep 3 14:26:33 UTC 2024


On Mon, 22 Jan 2024 09:23:05 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Quan Anh Mai has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   fix tests, add verify
>
> src/hotspot/share/opto/type.hpp line 558:
> 
>> 556: 
>> 557:   // Use to compute join of 2 sets
>> 558:   const bool _dual;
> 
> I think you need to add some comments, explaining why this is here

Thanks for adding the comment! I think comments should not refer to "before this change", rather they should describe the way the code works now.

I would also recommend renaming it to `_is_dual`. Because with `_dual`, I would expect to get some kind of dual, not a bool.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17508#discussion_r1742052610


More information about the hotspot-compiler-dev mailing list