* [PATCH] arm: Fix __arm_vctp16q return type in arm_mve.h
@ 2021-08-16 13:12 Christophe Lyon
2021-08-23 8:59 ` Christophe LYON
2021-08-23 9:30 ` Kyrylo Tkachov
0 siblings, 2 replies; 6+ messages in thread
From: Christophe Lyon @ 2021-08-16 13:12 UTC (permalink / raw)
To: gcc-patches
__arm_vctp16q actually returns mve_pred16_t rather than int64_t.
2021-08-16 Christophe Lyon <christophe.lyon@foss.st.com>
gcc/
* config/arm/arm_mve.h: Fix __arm_vctp16q return type.
---
gcc/config/arm/arm_mve.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/config/arm/arm_mve.h b/gcc/config/arm/arm_mve.h
index 83f10036990..e04d46218d0 100644
--- a/gcc/config/arm/arm_mve.h
+++ b/gcc/config/arm/arm_mve.h
@@ -3524,7 +3524,7 @@ __arm_vaddlvq_u32 (uint32x4_t __a)
return __builtin_mve_vaddlvq_uv4si (__a);
}
-__extension__ extern __inline int64_t
+__extension__ extern __inline mve_pred16_t
__attribute__ ((__always_inline__, __gnu_inline__, __artificial__))
__arm_vctp16q (uint32_t __a)
{
--
2.25.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm: Fix __arm_vctp16q return type in arm_mve.h
2021-08-16 13:12 [PATCH] arm: Fix __arm_vctp16q return type in arm_mve.h Christophe Lyon
@ 2021-08-23 8:59 ` Christophe LYON
2021-08-23 9:30 ` Kyrylo Tkachov
1 sibling, 0 replies; 6+ messages in thread
From: Christophe LYON @ 2021-08-23 8:59 UTC (permalink / raw)
To: gcc Patches
ping?
On 16/08/2021 15:12, Christophe Lyon wrote:
> __arm_vctp16q actually returns mve_pred16_t rather than int64_t.
>
> 2021-08-16 Christophe Lyon <christophe.lyon@foss.st.com>
>
> gcc/
> * config/arm/arm_mve.h: Fix __arm_vctp16q return type.
> ---
> gcc/config/arm/arm_mve.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/config/arm/arm_mve.h b/gcc/config/arm/arm_mve.h
> index 83f10036990..e04d46218d0 100644
> --- a/gcc/config/arm/arm_mve.h
> +++ b/gcc/config/arm/arm_mve.h
> @@ -3524,7 +3524,7 @@ __arm_vaddlvq_u32 (uint32x4_t __a)
> return __builtin_mve_vaddlvq_uv4si (__a);
> }
>
> -__extension__ extern __inline int64_t
> +__extension__ extern __inline mve_pred16_t
> __attribute__ ((__always_inline__, __gnu_inline__, __artificial__))
> __arm_vctp16q (uint32_t __a)
> {
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH] arm: Fix __arm_vctp16q return type in arm_mve.h
2021-08-16 13:12 [PATCH] arm: Fix __arm_vctp16q return type in arm_mve.h Christophe Lyon
2021-08-23 8:59 ` Christophe LYON
@ 2021-08-23 9:30 ` Kyrylo Tkachov
2021-08-23 9:41 ` Christophe LYON
1 sibling, 1 reply; 6+ messages in thread
From: Kyrylo Tkachov @ 2021-08-23 9:30 UTC (permalink / raw)
To: Christophe Lyon; +Cc: gcc-patches
> -----Original Message-----
> From: Gcc-patches <gcc-patches-
> bounces+kyrylo.tkachov=arm.com@gcc.gnu.org> On Behalf Of Christophe
> Lyon via Gcc-patches
> Sent: 16 August 2021 14:13
> To: gcc-patches@gcc.gnu.org
> Subject: [PATCH] arm: Fix __arm_vctp16q return type in arm_mve.h
>
> __arm_vctp16q actually returns mve_pred16_t rather than int64_t.
>
Ok. Does this need a backport?
Thanks,
Kyrill
> 2021-08-16 Christophe Lyon <christophe.lyon@foss.st.com>
>
> gcc/
> * config/arm/arm_mve.h: Fix __arm_vctp16q return type.
> ---
> gcc/config/arm/arm_mve.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/config/arm/arm_mve.h b/gcc/config/arm/arm_mve.h
> index 83f10036990..e04d46218d0 100644
> --- a/gcc/config/arm/arm_mve.h
> +++ b/gcc/config/arm/arm_mve.h
> @@ -3524,7 +3524,7 @@ __arm_vaddlvq_u32 (uint32x4_t __a)
> return __builtin_mve_vaddlvq_uv4si (__a);
> }
>
> -__extension__ extern __inline int64_t
> +__extension__ extern __inline mve_pred16_t
> __attribute__ ((__always_inline__, __gnu_inline__, __artificial__))
> __arm_vctp16q (uint32_t __a)
> {
> --
> 2.25.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm: Fix __arm_vctp16q return type in arm_mve.h
2021-08-23 9:30 ` Kyrylo Tkachov
@ 2021-08-23 9:41 ` Christophe LYON
2021-08-23 10:10 ` Kyrylo Tkachov
0 siblings, 1 reply; 6+ messages in thread
From: Christophe LYON @ 2021-08-23 9:41 UTC (permalink / raw)
To: Kyrylo Tkachov; +Cc: gcc-patches
On 23/08/2021 11:30, Kyrylo Tkachov wrote:
>
>> -----Original Message-----
>> From: Gcc-patches <gcc-patches-
>> bounces+kyrylo.tkachov=arm.com@gcc.gnu.org> On Behalf Of Christophe
>> Lyon via Gcc-patches
>> Sent: 16 August 2021 14:13
>> To: gcc-patches@gcc.gnu.org
>> Subject: [PATCH] arm: Fix __arm_vctp16q return type in arm_mve.h
>>
>> __arm_vctp16q actually returns mve_pred16_t rather than int64_t.
>>
> Ok. Does this need a backport?
Good point. I think so: branches gcc-10 and gcc-11 are affected, gcc-9
did not support MVE yet.
> Thanks,
> Kyrill
>
>> 2021-08-16 Christophe Lyon <christophe.lyon@foss.st.com>
>>
>> gcc/
>> * config/arm/arm_mve.h: Fix __arm_vctp16q return type.
>> ---
>> gcc/config/arm/arm_mve.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/gcc/config/arm/arm_mve.h b/gcc/config/arm/arm_mve.h
>> index 83f10036990..e04d46218d0 100644
>> --- a/gcc/config/arm/arm_mve.h
>> +++ b/gcc/config/arm/arm_mve.h
>> @@ -3524,7 +3524,7 @@ __arm_vaddlvq_u32 (uint32x4_t __a)
>> return __builtin_mve_vaddlvq_uv4si (__a);
>> }
>>
>> -__extension__ extern __inline int64_t
>> +__extension__ extern __inline mve_pred16_t
>> __attribute__ ((__always_inline__, __gnu_inline__, __artificial__))
>> __arm_vctp16q (uint32_t __a)
>> {
>> --
>> 2.25.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH] arm: Fix __arm_vctp16q return type in arm_mve.h
2021-08-23 9:41 ` Christophe LYON
@ 2021-08-23 10:10 ` Kyrylo Tkachov
2021-08-23 12:04 ` Christophe LYON
0 siblings, 1 reply; 6+ messages in thread
From: Kyrylo Tkachov @ 2021-08-23 10:10 UTC (permalink / raw)
To: Christophe LYON; +Cc: gcc-patches
> -----Original Message-----
> From: Christophe LYON <christophe.lyon@foss.st.com>
> Sent: 23 August 2021 10:41
> To: Kyrylo Tkachov <Kyrylo.Tkachov@arm.com>
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH] arm: Fix __arm_vctp16q return type in arm_mve.h
>
>
> On 23/08/2021 11:30, Kyrylo Tkachov wrote:
> >
> >> -----Original Message-----
> >> From: Gcc-patches <gcc-patches-
> >> bounces+kyrylo.tkachov=arm.com@gcc.gnu.org> On Behalf Of
> Christophe
> >> Lyon via Gcc-patches
> >> Sent: 16 August 2021 14:13
> >> To: gcc-patches@gcc.gnu.org
> >> Subject: [PATCH] arm: Fix __arm_vctp16q return type in arm_mve.h
> >>
> >> __arm_vctp16q actually returns mve_pred16_t rather than int64_t.
> >>
> > Ok. Does this need a backport?
>
>
> Good point. I think so: branches gcc-10 and gcc-11 are affected, gcc-9
> did not support MVE yet.
Ok, please backport it wherever it's needed.
Thanks,
Kyrill
>
>
> > Thanks,
> > Kyrill
> >
> >> 2021-08-16 Christophe Lyon <christophe.lyon@foss.st.com>
> >>
> >> gcc/
> >> * config/arm/arm_mve.h: Fix __arm_vctp16q return type.
> >> ---
> >> gcc/config/arm/arm_mve.h | 2 +-
> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/gcc/config/arm/arm_mve.h b/gcc/config/arm/arm_mve.h
> >> index 83f10036990..e04d46218d0 100644
> >> --- a/gcc/config/arm/arm_mve.h
> >> +++ b/gcc/config/arm/arm_mve.h
> >> @@ -3524,7 +3524,7 @@ __arm_vaddlvq_u32 (uint32x4_t __a)
> >> return __builtin_mve_vaddlvq_uv4si (__a);
> >> }
> >>
> >> -__extension__ extern __inline int64_t
> >> +__extension__ extern __inline mve_pred16_t
> >> __attribute__ ((__always_inline__, __gnu_inline__, __artificial__))
> >> __arm_vctp16q (uint32_t __a)
> >> {
> >> --
> >> 2.25.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm: Fix __arm_vctp16q return type in arm_mve.h
2021-08-23 10:10 ` Kyrylo Tkachov
@ 2021-08-23 12:04 ` Christophe LYON
0 siblings, 0 replies; 6+ messages in thread
From: Christophe LYON @ 2021-08-23 12:04 UTC (permalink / raw)
To: Kyrylo Tkachov; +Cc: gcc-patches
On 23/08/2021 12:10, Kyrylo Tkachov wrote:
>
>> -----Original Message-----
>> From: Christophe LYON <christophe.lyon@foss.st.com>
>> Sent: 23 August 2021 10:41
>> To: Kyrylo Tkachov <Kyrylo.Tkachov@arm.com>
>> Cc: gcc-patches@gcc.gnu.org
>> Subject: Re: [PATCH] arm: Fix __arm_vctp16q return type in arm_mve.h
>>
>>
>> On 23/08/2021 11:30, Kyrylo Tkachov wrote:
>>>> -----Original Message-----
>>>> From: Gcc-patches <gcc-patches-
>>>> bounces+kyrylo.tkachov=arm.com@gcc.gnu.org> On Behalf Of
>> Christophe
>>>> Lyon via Gcc-patches
>>>> Sent: 16 August 2021 14:13
>>>> To: gcc-patches@gcc.gnu.org
>>>> Subject: [PATCH] arm: Fix __arm_vctp16q return type in arm_mve.h
>>>>
>>>> __arm_vctp16q actually returns mve_pred16_t rather than int64_t.
>>>>
>>> Ok. Does this need a backport?
>>
>> Good point. I think so: branches gcc-10 and gcc-11 are affected, gcc-9
>> did not support MVE yet.
> Ok, please backport it wherever it's needed.
> Thanks,
> Kyrill
Thanks, committed to trunk, gcc-11 and gcc-10.
Christophe
>>
>>> Thanks,
>>> Kyrill
>>>
>>>> 2021-08-16 Christophe Lyon <christophe.lyon@foss.st.com>
>>>>
>>>> gcc/
>>>> * config/arm/arm_mve.h: Fix __arm_vctp16q return type.
>>>> ---
>>>> gcc/config/arm/arm_mve.h | 2 +-
>>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/gcc/config/arm/arm_mve.h b/gcc/config/arm/arm_mve.h
>>>> index 83f10036990..e04d46218d0 100644
>>>> --- a/gcc/config/arm/arm_mve.h
>>>> +++ b/gcc/config/arm/arm_mve.h
>>>> @@ -3524,7 +3524,7 @@ __arm_vaddlvq_u32 (uint32x4_t __a)
>>>> return __builtin_mve_vaddlvq_uv4si (__a);
>>>> }
>>>>
>>>> -__extension__ extern __inline int64_t
>>>> +__extension__ extern __inline mve_pred16_t
>>>> __attribute__ ((__always_inline__, __gnu_inline__, __artificial__))
>>>> __arm_vctp16q (uint32_t __a)
>>>> {
>>>> --
>>>> 2.25.1
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-08-23 12:04 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-16 13:12 [PATCH] arm: Fix __arm_vctp16q return type in arm_mve.h Christophe Lyon
2021-08-23 8:59 ` Christophe LYON
2021-08-23 9:30 ` Kyrylo Tkachov
2021-08-23 9:41 ` Christophe LYON
2021-08-23 10:10 ` Kyrylo Tkachov
2021-08-23 12:04 ` Christophe LYON
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).