On Wed, 7 Apr 2021 09:50:45 GMT, Andrew Haley <a...@openjdk.org> wrote:

>> Dong Bo has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   fix misleading annotations
>
> src/hotspot/cpu/aarch64/stubGenerator_aarch64.cpp line 5829:
> 
>> 5827:     __ strb(r14, __ post(dst, 1));
>> 5828:     __ strb(r15, __ post(dst, 1));
>> 5829:     __ strb(r13, __ post(dst, 1));
> 
> I think this sequence should be 4 BFMs, STRW, BFM, STRW. That's the best we 
> can do, I think.

Sorry, that's not quite right, but you get the idea: let's not generate 
unnecessary memory traffic.

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

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

Reply via email to