RFR: 8369569: Rename methods in regmask.hpp to conform with HotSpot coding style

Emanuel Peter epeter at openjdk.org
Fri Oct 17 12:54:05 UTC 2025


On Wed, 15 Oct 2025 08:14:26 GMT, Daniel Lundén <dlunden at openjdk.org> wrote:

> A number of methods in regmask.hpp do not conform with the HotSpot coding style. We should make sure they do.
> 
> ### Changeset
> - Rename methods in `regmask.hpp` to conform with HotSpot coding style.
> - Similarly rename directly related methods in `chaitin.hpp`.
> - Rename the constant register masks `All` and `Empty` to `ALL` and `EMPTY`.
> - Fix a few additional code style issues at lines touched by the changeset.
> 
> Note: this is a syntax-only changeset (no functional changes).
> 
> ### Testing
> 
> - [GitHub Actions](https://github.com/dlunde/jdk/actions/runs/18500704336)
> - `tier1` and HotSpot parts of `tier2` and `tier3` (and additional Oracle-internal testing) on Windows x64, Linux x64, Linux aarch64, macOS x64, and macOS aarch64.

LGTM. Roberto had some good suggestions though, so you should look at those :)

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

Marked as reviewed by epeter (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/27817#pullrequestreview-3350205643


More information about the hotspot-compiler-dev mailing list