RFR: 8354196: C2: reorder and capitalize phase definition [v2]

Thu Jun 12 12:54:32 UTC 2025

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

>> src/hotspot/share/opto/phasetype.hpp line 54:
>> 
>>> 52:   flags(PHASEIDEAL_BEFORE_EA,           "PhaseIdealLoop before EA") \
>>> 53:   flags(AFTER_EA,                       "After Escape Analysis") \
>>> 54:   flags(ITER_GVN_AFTER_EA,              "Iter GVN after EA") \
>> 
>> Not sure if this should be also part of this change but we might want to consider "Iter GVN" -> IGVN (same for left hand sides of `flags()`.
>
> I will comment on JDK-8319599 to propose it.

Sounds good, thanks!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25778#discussion_r2142661389