RFR: 8338727: RISC-V: Avoid synthetic data dependency in nmethod barrier on Ztso [v2]
Hamlin Li
mli at openjdk.org
Tue Aug 27 08:12:03 UTC 2024
On Mon, 26 Aug 2024 06:24:15 GMT, Robbin Ehn <rehn at openjdk.org> wrote:
>> Hi please consider,
>>
>> On TSO we don't need the synthetic data dependency in between the loads.
>> Also added some comment about this.
>>
>> Sanity tested
>
> Robbin Ehn has updated the pull request incrementally with one additional commit since the last revision:
>
> Comment update
Marked as reviewed by mli (Reviewer).
Yeah, the new process is bit tedious, espcially for additional minor change after one approval.
Maybe we can improve it a bit ourself in riscv area? For minor changes after approval, we can skip re-review/approve?
-------------
PR Review: https://git.openjdk.org/jdk/pull/20661#pullrequestreview-2262498987
PR Comment: https://git.openjdk.org/jdk/pull/20661#issuecomment-2311845544
More information about the hotspot-dev
mailing list