RFR: 8254264: Remove redundant cross_modify_fence()

Alan Hayward github.com+4146708+a74nh at openjdk.java.net
Tue Oct 20 08:16:23 UTC 2020


On Tue, 20 Oct 2020 01:25:10 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:

>>> Why is the commit history only showing a single commit if there was a merge in the past?
>>> (I might be missing something trivial here since I'm still a GitHub newbie.)
>> 
>> Not sure, where are you seeing the merge?
>
> Sorry Patricio. I saw the comment from a74nh above ("I merged this patch together with...")
> and I just assumed it was you replying to David H. My mistake.

> > I merged this patch together with #428 (which makes AArch64 use cross modify fence) and ran it on AArch64 with the
> > "-XX:+VerifyCrossModifyFence" set. A complete tier1 ran successfully, and I'm waiting on the results of a much wider
> > run.
> 
> Ok, let me know when you are okay and I will integrate.

I've run this across all our tests and machines now, and everything passes ok.

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

PR: https://git.openjdk.java.net/jdk/pull/655


More information about the hotspot-runtime-dev mailing list