RFR: 8371536: C2: VerifyIterativeGVN should assert on first detected failure [v9]

Benoît Maillard bmaillard at openjdk.org
Wed Jan 28 17:11:23 UTC 2026


On Wed, 28 Jan 2026 10:08:25 GMT, Manuel Hässig <mhaessig at openjdk.org> wrote:

>> Defining a small, say `GVNVerificationPhase`, enum would already be cleaner, safer and less overhead.
>
> You could even make that a debug only field of `PhaseIterGVN` and override it for `PhaseCCP`.

Following a discussion with @mhaessig, I decided to replace the `_iterGVN` boolean field by an enum field, and refactor `is_IterGVN` using this new field. 

```c++
enum class PhaseValuesType {
    gvn,
    iter_gvn,
    ccp
  };

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28295#discussion_r2737650119


More information about the hotspot-compiler-dev mailing list