Integrated: 8354196: C2: reorder and capitalize phase definition

Manuel Hässig mhaessig at openjdk.org
Fri Jun 13 13:05:38 UTC 2025


On Thu, 12 Jun 2025 11:10:38 GMT, Manuel Hässig <mhaessig at openjdk.org> wrote:

> This PR performs some cleanup and formatting around the phase definitions in C2:
>  - the phase descriptions are capitalized according to the [MLA Handbook title case rules](https://en.wikipedia.org/wiki/Title_case#Modern_Language_Association_(MLA)_Handbook),
>  - the phases are reordered to be more or less in the order of execution or occurrence in the code,
>  - the definitions in `phasetype.hpp` and `CompilePhase.java` are synced,
>  - `CompilePhase.java` is aligned for better readability.
> 
> This change was tested on:
>  - [x] [Github Actions](https://github.com/mhaessig/jdk/actions/runs/15605662671)
>  - [x] tier1 plus some Oracle internal testing

This pull request has now been integrated.

Changeset: b4c4496e
Author:    Manuel Hässig <mhaessig at openjdk.org>
Committer: Christian Hagedorn <chagedorn at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/b4c4496ef8013df25b6368bdebf082d223d6afed
Stats:     147 lines in 2 files changed: 28 ins; 27 del; 92 mod

8354196: C2: reorder and capitalize phase definition

Reviewed-by: chagedorn, mchevalier

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

PR: https://git.openjdk.org/jdk/pull/25778


More information about the hotspot-compiler-dev mailing list