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