Kyrylo Tkachov <kyrylo.tkac...@arm.com> writes:

> Hi Andrea,
>
>> -----Original Message-----
>> From: Gcc-patches <gcc-patches-boun...@gcc.gnu.org> On Behalf Of
>> Richard Sandiford
>> Sent: 21 September 2020 11:58
>> To: Andrea Corallo <andrea.cora...@arm.com>
>> Cc: Richard Earnshaw <richard.earns...@arm.com>; nd <n...@arm.com>;
>> gcc-patches@gcc.gnu.org
>> Subject: Re: [PATCH] aarch64: Do not alter value on a force_reg returned rtx
>> expanding __jcvt
>>
>> Andrea Corallo <andrea.cora...@arm.com> writes:
>> > Hi all,
>> >
>> > From the `force_reg` description comment I see the returned register
>> > should not be modified, thus IIUC should not be used as a GEN_FCN
>> > target.
>> >
>> > Assuming my interpretation is correct this fix this case inside
>> > `aarch64_general_expand_builtin` while expanding expanding the
>> > `__jcvt` intrinsic.  If is not the case please discard.
>>
>> Good catch.
>
> Can you please also backport it to the appropriate branches as well after 
> some time on trunk.
> Thanks,
> Kyrill

Ciao Kyrill,

Sure happy to do that.  For now into trunk as 2c62952f816.

Thanks

  Andrea

Reply via email to