RFR: 8356751: IGV: clean up redundant field _should_send_method
Manuel Hässig
mhaessig at openjdk.org
Tue Jun 10 11:29:28 UTC 2025
On Tue, 10 Jun 2025 08:58:15 GMT, Benoît Maillard <duke at openjdk.org> wrote:
> This PR removes private field `IdealGraphPrinter::_should_send_method` (as it was only ever assigned `true`) and all usages in conditions.
>
> ### Testing
>
> - [x] [GitHub Actions](https://github.com/benoitmaillard/jdk/actions?query=branch%3AJDK-8356751)
> - [x] tier1, and some Oracle internal testing
> - [x] Ran IdealGraphVisualizer and made sure that no crash occured
>
> Shout out to @mhaessig for introducing me to IGV
>
> Thanks!
Thank you for working on this and also for the credit :slightly_smiling_face:
This looks good to me.
-------------
Marked as reviewed by mhaessig (Author).
PR Review: https://git.openjdk.org/jdk/pull/25714#pullrequestreview-2913133373
More information about the hotspot-compiler-dev
mailing list