RFR: 8361144: Strenghten the Ideal Verification in PhaseIterGVN::verify_Ideal_for by comparing the hash of a node before and after Ideal [v2]

Benoît Maillard bmaillard at openjdk.org
Wed Jul 2 10:03:23 UTC 2025


> This PR adds a node hash comparison after calling `Ideal` in `PhaseIterGVN::verify_Ideal_for` to introduce an additional layer of verification for missed optimizations. Previously, we relied on the return value of `Ideal`, which is expected to be `nullptr` if no transformation was done.
> 
> By also checking the node's hash before and after `Ideal`, we could catch inconsistencies in the implementation or unintended modifications to the graph. Both of these may indicate missed or incomplete optimizations.
> 
> ### Testing
> - [x] [GitHub Actions](https://github.com/benoitmaillard/jdk/actions?query=branch%3AJDK-8361144)
> - [x] tier1-3, plus some internal testing
> 
> Thank you for reviewing!

Benoît Maillard has updated the pull request incrementally with one additional commit since the last revision:

  8361144: update comment
  
  Co-authored-by: Damon Fenacci <damon.fenacci at oracle.com>

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/26064/files
  - new: https://git.openjdk.org/jdk/pull/26064/files/e06b4d53..28851936

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=26064&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=26064&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/26064.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/26064/head:pull/26064

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


More information about the hotspot-compiler-dev mailing list