[aarch64-port-dev ] RFR: JDK-8196064: AArch64: Merging ld/st into ldp/stp in macro-assembler

[Zhongwei Yao]

Hi, Felix,

Thank for your review!

The change in zero_memory is necessary. Otherwise, the generated
multiple str instructions will be merged into stp, which breaks the
assumption of zero_memory's algorithm: each store instruction stores
one word. Check the generated assembly of zero_memory function may
help understand it better.

On 17 February 2018 at 09:15, Felix Yang <felix.yang at linaro.org> wrote:
> Hi,
>
>     Overall, the patch looks good to me.
>     One question: is the change in MacroAssembler::zero_memory necessary?
>
> Thanks,
> Felix
>
>
> On 12 February 2018 at 18:03, Zhongwei Yao <zhongwei.yao at linaro.org> wrote:
>>
>> I see. Thanks.
>>
>> On 12 February 2018 at 17:51, Andrew Haley <aph at redhat.com> wrote:
>> > On 11/02/18 02:56, Zhongwei Yao wrote:
>> >> And I forgot to mention that since this patch touches some shared code
>> >> at hotspot/share/asm/codeBuffer.hpp, do I need a sponsor from Oracle?
>> >
>> > In theory, no.  The submission forest should do that job.  I'll have a
>> > try.
>> >
>> > --
>> > Andrew Haley
>> > Java Platform Lead Engineer
>> > Red Hat UK Ltd. <https://www.redhat.com>
>> > EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671
>>
>>
>>
>> --
>> Best regards,
>> Zhongwei
>
>



-- 
Best regards,
Zhongwei