RFR: 8367397: Improve naming and terminology in regmask.hpp and regmask.cpp [v2]

Emanuel Peter epeter at openjdk.org
Fri Sep 12 05:52:12 UTC 2025


On Fri, 12 Sep 2025 00:08:06 GMT, Dean Long <dlong at openjdk.org> wrote:

>> Daniel Lundén has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Lowercase _RM_INT and _RM_WORD
>
> src/hotspot/share/opto/regmask.hpp line 66:
> 
>> 64: 
>> 65:   static const unsigned int _WordBitMask = BitsPerWord - 1U;
>> 66:   static const unsigned int _LogWordBits = LogBitsPerWord;
> 
> What about just replacing all uses of _LogWordBits with LogBitsPerWord?

Yes, that would be a good step in the right direction.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27215#discussion_r2343046545


More information about the hotspot-compiler-dev mailing list