RFR: 8305236: Some LoadLoad barriers in the interpreter are unnecessary after JDK-8220051
SUN Guoyun
duke at openjdk.org
Tue Apr 4 06:28:11 UTC 2023
On Tue, 4 Apr 2023 02:23:01 GMT, David Holmes <dholmes 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.
>
> May I suggest changing the PR and JBS issue title to:
>
> 8305236: Some LoadLoad barriers in the interpreter are unnecessary after JDK-8220051
Thanks @dholmes-ora to review. @theRealAph @RealFYang Please help confirm if there is still a problem with the patch.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/13244#issuecomment-1495417609
More information about the hotspot-runtime-dev
mailing list