RFR: JDK-8305644: IGV: Node text not updated when switching from/to CFG view [v2]
Christian Hagedorn
chagedorn at openjdk.org
Wed Apr 5 12:00:10 UTC 2023
On Wed, 5 Apr 2023 11:53:05 GMT, Tobias Holenstein <tholenstein at openjdk.org> wrote:
>> When switching the layouting mode to or from CFG the node text is not updated.
>>
>> - Switching to `CFG view` gave the wrong node text:
>> <img width="487" alt="fail_to" src="https://user-images.githubusercontent.com/71546117/230041157-9f6613ef-112c-44d6-8263-63f33c9e56e3.png">
>> - with this fix it looks like this
>> <img width="514" alt="fix_to" src="https://user-images.githubusercontent.com/71546117/230041139-09f10865-958c-415c-96fe-b036cc2c061a.png">
>>
>> - Switching from `CFG view` gave the wrong node text:
>> <img width="682" alt="fail_from" src="https://user-images.githubusercontent.com/71546117/230041151-7c56e33e-7fd2-46cb-b73d-70a7d194f666.png">
>> - with this fix it looks like this
>> <img width="599" alt="fix-from" src="https://user-images.githubusercontent.com/71546117/230041147-734d1fd0-507b-462e-a4d2-d8b039f333dc.png">
>
> Tobias Holenstein has updated the pull request incrementally with one additional commit since the last revision:
>
> fix the bug properly
That looks much better! Thanks for investigating further how to properly fix this :-)
-------------
Marked as reviewed by chagedorn (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/13348#pullrequestreview-1372753641
More information about the hotspot-compiler-dev
mailing list