RFR: 8354196: C2: reorder and capitalize phase definition [v2]
Manuel Hässig
mhaessig at openjdk.org
Thu Jun 12 12:20:16 UTC 2025
On Thu, 12 Jun 2025 11:41:36 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:
>> Manuel Hässig has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Split-If is a proper name
>
> test/hotspot/jtreg/compiler/lib/ir_framework/CompilePhase.java line 73:
>
>> 71: AFTER_LOOP_UNROLLING( "After Loop Unrolling"),
>> 72: BEFORE_SPLIT_IF( "Before Split-if"),
>> 73: AFTER_SPLIT_IF( "After Split-if"),
>
> Nit: We might want to name it "Split-If" as being a name? At least we also call it as such in `TraceLoopOpts`:
> https://github.com/openjdk/jdk/blob/65e63b6ab4241fc9d683e2ffa5bfe6e1a30059b6/src/hotspot/share/opto/loopopts.cpp#L1465
> (same for `phasetype.hpp`)
I like the proper name argument. Fixed in 9f3676e4a
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25778#discussion_r2142583277
More information about the hotspot-compiler-dev
mailing list