Hi Richard
> -Original Message-
> From: Richard Sandiford
> Sent: 11 November 2020 17:52
> To: Sudakshina Das
> Cc: Wilco Dijkstra ; gcc-patches@gcc.gnu.org;
> Kyrylo Tkachov ; Richard Earnshaw
>
> Subject: Re: [PATCH] aarch64: Add backend support for expa
Sudakshina Das writes:
> Apologies for the delay. I have attached another version of the patch.
> I have disabled the test cases for ILP32. This is only because function body
> check
> fails because there is an addition unsigned extension instruction for src
> pointer in
> every test (uxtwx0
Hi Richard
> -Original Message-
> From: Richard Sandiford
> Sent: 03 November 2020 11:34
> To: Sudakshina Das
> Cc: Wilco Dijkstra ; gcc-patches@gcc.gnu.org;
> Kyrylo Tkachov ; Richard Earnshaw
>
> Subject: Re: [PATCH] aarch64: Add backend support for expa
Sudakshina Das writes:
>> -Original Message-
>> From: Richard Sandiford
>> Sent: 30 October 2020 19:56
>> To: Sudakshina Das
>> Cc: Wilco Dijkstra ; gcc-patches@gcc.gnu.org;
>> Kyrylo Tkachov ; Richard Earnshaw
>>
>> Subject: Re: [P
Hi Richard
> -Original Message-
> From: Richard Sandiford
> Sent: 30 October 2020 19:56
> To: Sudakshina Das
> Cc: Wilco Dijkstra ; gcc-patches@gcc.gnu.org;
> Kyrylo Tkachov ; Richard Earnshaw
>
> Subject: Re: [PATCH] aarch64: Add backend support for expa
Sudakshina Das writes:
>> > +
>> > + /* "Cast" the *dst to the correct mode. */
>> > + *dst = adjust_address (*dst, mode, 0);
>> > + /* Emit the memset. */
>> > + emit_move_insn (*dst, reg);
>> > + /* Move the pointer forward. */
>> > + *dst = aarch64_progress_pointer (*dst); }
>> > +
>>
Hi Richard
Thank you for the review. Please find my comments inlined.
> -Original Message-
> From: Richard Sandiford
> Sent: 30 October 2020 15:03
> To: Sudakshina Das
> Cc: gcc-patches@gcc.gnu.org; Kyrylo Tkachov ;
> Richard Earnshaw
> Subject: Re: [PATCH]
Sudakshina Das writes:
> diff --git a/gcc/config/aarch64/aarch64.h b/gcc/config/aarch64/aarch64.h
> index
> 00b5f8438863bb52c348cfafd5d4db478fe248a7..bcb654809c9662db0f51fc1368e37e42969efd29
> 100644
> --- a/gcc/config/aarch64/aarch64.h
> +++ b/gcc/config/aarch64/aarch64.h
> @@ -1024,16 +1024,18
Hi
This patch implements aarch64 backend expansion for __builtin_memset. Most of
the
implementation is based on the expansion of __builtin_memcpy. We change the
values of
SET_RATIO and MOVE_RATIO for cases where we do not have to strictly align and
where
we can benefit from NEON instructions in
