RFR: 8305236: Some LoadLoad barriers in the interpreter are unnecessary after JDK-8220051 [v3]

Tue Apr 4 07:13:09 UTC 2023

On Tue, 4 Apr 2023 05:13:09 GMT, SUN Guoyun <duke at openjdk.org> wrote:

>> After JDK-8220051, Interpreter::notice_safepoints() only be executed at a safe point, so LoadLoad barrier is useless.
>> Barrier directives are generally time-consuming, so this patch removes LoadLoad be used for aarch64 and riscv.
>> 
>> Jtreg tier1-2 testing for aarch64 has been done.
>> 
>> Please help review it.
>> 
>> Thanks.
>
> SUN Guoyun has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains one commit:
> 
>   8305236: Some LoadLoad barriers in the interpreter are unnecessary after JDK-8220051

That looks reasonable to me. I am performing some tests on linux-riscv64 platform.

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

PR Review: https://git.openjdk.org/jdk/pull/13244#pullrequestreview-1370335050