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