On Thu, 23 Mar 2023 15:06:21 GMT, Matias Saavedra Silva <matsa...@openjdk.org> wrote:
>> Matias Saavedra Silva has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Improved comment for load-acquire aarch64 > > src/hotspot/cpu/ppc/templateTable_ppc_64.cpp line 2294: > >> 2292: >> 2293: __ load_resolved_indy_entry(cache, index); >> 2294: __ ld_ptr(method, in_bytes(ResolvedIndyEntry::method_offset()), >> cache); > > @reinrich this load needs acquire semantics. We already have ordering by `isync(); // Order load wrt. succeeding loads.` below. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/12778#discussion_r1146353107
