Integrated: 8299312: Clean up BarrierSetNMethod

Mon Jan 9 13:38:00 UTC 2023

On Fri, 23 Dec 2022 12:00:46 GMT, Erik Österlund <eosterlund at openjdk.org> wrote:

> The terminology in BarrierSetNMethod is not crisp. In platform code we talk about a per-nmethod "guard value", but on shared level we call the same value arm value or disarm value in different contexts. But it really depends on the value whether the nmethod is disarmed or armed. We should embrace the "guard value" terminology and lift it in to the shared code level.
> We also have more functionality than we need on platform level. The platform level only needs to know how to deoptimize, and how to set/get the guard value of an nmethod. The more specific functionality should be moved to the shared code and be expressed in terms of said setter/getter.

This pull request has now been integrated.

Changeset: 4ba81221
Author:    Erik Österlund <eosterlund at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/4ba8122197e912db4894ed7fe395a8842268fbef
Stats:     175 lines in 29 files changed: 10 ins; 82 del; 83 mod

8299312: Clean up BarrierSetNMethod

Reviewed-by: mdoerr, fyang

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

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