Integrated: 8282480: IGV: Use description instead of enum name for phases

Christian Hagedorn chagedorn at openjdk.java.net
Wed Mar 2 09:01:05 UTC 2022


On Tue, 1 Mar 2022 13:03:50 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

> [JDK-8281505](https://bugs.openjdk.java.net/browse/JDK-8281505) changed the `PrintIdeal` output to use the enum name of a phase when used with `PrintIdealPhase` instead of the description. However, it also changed the phase name shown in the IGV to the enum name. This should be reverted back for a better readability.
> 
> IGV after JDK-8281505:
> ![Screenshot from 2022-03-01 13-59-15](https://user-images.githubusercontent.com/17833009/156173248-7f6450cb-9816-437a-b5fa-816a2e214ad9.png)
> 
> IGV before JDK-8281505/with this patch:
> ![Screenshot from 2022-03-01 13-58-36](https://user-images.githubusercontent.com/17833009/156173243-0a0f975a-a44d-46a6-a48a-5183ff7600e1.png)
> 
> Thanks,
> Christian

This pull request has now been integrated.

Changeset: ed3496e6
Author:    Christian Hagedorn <chagedorn at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/ed3496e6c030b6b0a3745bf8ef281075afa1cd27
Stats:     1 line in 1 file changed: 0 ins; 0 del; 1 mod

8282480: IGV: Use description instead of enum name for phases

Reviewed-by: redestad, thartmann

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

PR: https://git.openjdk.java.net/jdk/pull/7645


More information about the hotspot-compiler-dev mailing list