Skip to content

8357424: [JVMCI] Avoid incrementing decompilation count for hosted compiled nmethod #25356

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 12 commits into from
Closed

8357424: [JVMCI] Avoid incrementing decompilation count for hosted compiled nmethod #25356

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
8fcd710
[JVMCI] Avoid incrementing decompilation count for hosted compiled nm…
mur47x111 May 21, 2025
ef4a4c9
update copyright
mur47x111 May 21, 2025
e66c16d
address comments
mur47x111 May 22, 2025
b72213a
address comments
mur47x111 May 22, 2025
e508dec
Merge remote-tracking branch 'upstream/master' into JDK-8357424
mur47x111 Jun 10, 2025
57fe530
address comments
mur47x111 Jun 11, 2025
9d24428
fix compilation error
mur47x111 Jun 11, 2025
f253c0a
Merge tag 'jdk-26+2' into JDK-8357424
mur47x111 Jun 13, 2025
fb32a8c
Merge tag 'jdk-26+3' into JDK-8357424
mur47x111 Jun 30, 2025
fc7a7cb
Merge remote-tracking branch 'upstream/master' into JDK-8357424
mur47x111 Jul 2, 2025
022546a
address comments
mur47x111 Jul 2, 2025
d622192
address comments
mur47x111 Jul 2, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions src/hotspot/share/code/nmethod.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1936,9 +1936,9 @@ void nmethod::inc_decompile_count() {
if (!is_compiled_by_c2() && !is_compiled_by_jvmci()) return;
// Could be gated by ProfileTraps, but do not bother...
#if INCLUDE_JVMCI
// Deoptimizations from non-default (non-CompileBroker) compilations should not
// affect the Java method's profile, as they could block it from being compiled
// at the highest tier.
// Deoptimization count is used by the CompileBroker to reason about compilations
// it requests so do not pollute the count for deoptimizations in non-default (i.e.
// non-CompilerBroker) compilations.
if (is_jvmci_hosted()) {
return;
}
Expand Down
6 changes: 3 additions & 3 deletions src/hotspot/share/runtime/deoptimization.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2364,9 +2364,9 @@ JRT_ENTRY(void, Deoptimization::uncommon_trap_inner(JavaThread* current, jint tr
}

#if INCLUDE_JVMCI
// Deoptimizations from non-default (non-CompileBroker) compilations should not
// affect the Java method's profile, as they could block it from being compiled
// at the highest tier.
// Deoptimization count is used by the CompileBroker to reason about compilations
// it requests so do not pollute the count for deoptimizations in non-default (i.e.
// non-CompilerBroker) compilations.
if (nm->is_jvmci_hosted()) {
update_trap_state = false;
}
Expand Down