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