RFR: 8273380: ARM32: Default to {ldrexd, strexd} in StubRoutines::atomic_{load|store}_long [v2]

Dmitry Samersoff dsamersoff at openjdk.java.net
Fri Sep 24 14:27:05 UTC 2021


On Thu, 23 Sep 2021 15:54:33 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> src/hotspot/cpu/arm/stubGenerator_arm.cpp line 641:
>> 
>>> 639:       __ ldrexd(result_lo, Address(src));
>>> 640:       __ clrex(); // FIXME: safe to remove?
>>> 641:       __ bx(LR);
>> 
>> bx(LR) is common for all 3 branches, so it might be better to move it out
>
> @dsamersoff, are you happy with new version?

Yes. Thank you!

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

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


More information about the hotspot-compiler-dev mailing list