Integrated: 8356751: IGV: clean up redundant field _should_send_method

Benoît Maillard duke at openjdk.org
Wed Jun 11 11:11:37 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!

This pull request has now been integrated.

Changeset: bf7d40d0
Author:    Benoît Maillard <benoit.maillard at oracle.com>
Committer: Damon Fenacci <dfenacci at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/bf7d40d0486b7b4e4820bb5d08a63c446ea3291d
Stats:     11 lines in 2 files changed: 0 ins; 5 del; 6 mod

8356751: IGV: clean up redundant field _should_send_method

Co-authored-by: Manuel Hässig <mhaessig at openjdk.org>
Reviewed-by: mhaessig, thartmann, dfenacci

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

PR: https://git.openjdk.org/jdk/pull/25714


More information about the hotspot-compiler-dev mailing list