Integrated: 8350330: C2: PhaseIdealLoop::add_parse_predicate() should mirror GraphKit::add_parse_predicate()

Roland Westrelin roland at openjdk.org
Wed Jan 28 08:06:09 UTC 2026


On Thu, 22 Jan 2026 16:22:34 GMT, Roland Westrelin <roland at openjdk.org> wrote:

> `PhaseIdealLoop::add_parse_predicate()` was intented to mirror
> `GraphKit::add_parse_predicate()` but it doesn't. That last one checks
> `too_many_traps` per bci but the `PhaseIdealLoop` version doesn't. As
> demonstrated by the test case, a method can get compiled with a
> predicate, take a trap, and get recompiled with the same predicate
> many times (up to ~100).

This pull request has now been integrated.

Changeset: b2cd3b0d
Author:    Roland Westrelin <roland at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/b2cd3b0d48bdabacfd421dee9b9f87a003e0e09d
Stats:     123 lines in 3 files changed: 100 ins; 20 del; 3 mod

8350330: C2: PhaseIdealLoop::add_parse_predicate() should mirror GraphKit::add_parse_predicate()

Reviewed-by: chagedorn, qamai

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

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


More information about the hotspot-compiler-dev mailing list