* [PATCH] [aarch64] Revert support for ARMv8.2 in tsv110
@ 2018-12-19 3:12 Shaokun Zhang
2018-12-19 10:12 ` Richard Earnshaw (lists)
0 siblings, 1 reply; 3+ messages in thread
From: Shaokun Zhang @ 2018-12-19 3:12 UTC (permalink / raw)
To: gcc-patches
Cc: richard.earnshaw, james.greenhalgh, marcus.shawcroft,
richard.sandiford, kyrylo.tkachov, nickc, wuyuan5, Shaokun Zhang
For HiSilicon's tsv110 cpu core, it supports some v8_4A features, but
some mandatory features are not implemented. Revert to ARMv8.2 that
all mandatory features are supported.
---
gcc/ChangeLog | 5 +++++
gcc/config/aarch64/aarch64-cores.def | 6 +++---
2 files changed, 8 insertions(+), 3 deletions(-)
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index e9f5baa6557c..842876b0ae90 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2018-12-19 Shaokun Zhang <zhangshaokun@hisilicon.com>
+
+ * config/aarch64/aarch64-cores.def (tsv110) : Revert support for ARMv8.2
+ in tsv110.
+
2018-12-18 Vladimir Makarov <vmakarov@redhat.com>
PR rtl-optimization/87759
diff --git a/gcc/config/aarch64/aarch64-cores.def b/gcc/config/aarch64/aarch64-cores.def
index 74be5dbf2595..20f4924e084d 100644
--- a/gcc/config/aarch64/aarch64-cores.def
+++ b/gcc/config/aarch64/aarch64-cores.def
@@ -96,10 +96,10 @@ AARCH64_CORE("cortex-a75", cortexa75, cortexa57, 8_2A, AARCH64_FL_FOR_ARCH8_2
AARCH64_CORE("cortex-a76", cortexa76, cortexa57, 8_2A, AARCH64_FL_FOR_ARCH8_2 | AARCH64_FL_F16 | AARCH64_FL_RCPC | AARCH64_FL_DOTPROD, cortexa72, 0x41, 0xd0b, -1)
AARCH64_CORE("ares", ares, cortexa57, 8_2A, AARCH64_FL_FOR_ARCH8_2 | AARCH64_FL_F16 | AARCH64_FL_RCPC | AARCH64_FL_DOTPROD | AARCH64_FL_PROFILE, cortexa72, 0x41, 0xd0c, -1)
-/* ARMv8.4-A Architecture Processors. */
-
/* HiSilicon ('H') cores. */
-AARCH64_CORE("tsv110", tsv110, cortexa57, 8_4A, AARCH64_FL_FOR_ARCH8_4 | AARCH64_FL_CRYPTO | AARCH64_FL_F16 | AARCH64_FL_AES | AARCH64_FL_SHA2, tsv110, 0x48, 0xd01, -1)
+AARCH64_CORE("tsv110", tsv110, cortexa57, 8_2A, AARCH64_FL_FOR_ARCH8_2 | AARCH64_FL_CRYPTO | AARCH64_FL_F16 | AARCH64_FL_AES | AARCH64_FL_SHA2, tsv110, 0x48, 0xd01, -1)
+
+/* ARMv8.4-A Architecture Processors. */
/* Qualcomm ('Q') cores. */
AARCH64_CORE("saphira", saphira, saphira, 8_4A, AARCH64_FL_FOR_ARCH8_4 | AARCH64_FL_CRYPTO | AARCH64_FL_RCPC, saphira, 0x51, 0xC01, -1)
--
2.7.4
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] [aarch64] Revert support for ARMv8.2 in tsv110
2018-12-19 3:12 [PATCH] [aarch64] Revert support for ARMv8.2 in tsv110 Shaokun Zhang
@ 2018-12-19 10:12 ` Richard Earnshaw (lists)
2018-12-20 1:12 ` Zhangshaokun
0 siblings, 1 reply; 3+ messages in thread
From: Richard Earnshaw (lists) @ 2018-12-19 10:12 UTC (permalink / raw)
To: Shaokun Zhang, gcc-patches
Cc: james.greenhalgh, marcus.shawcroft, richard.sandiford,
kyrylo.tkachov, nickc, wuyuan5
On 19/12/2018 03:11, Shaokun Zhang wrote:
> For HiSilicon's tsv110 cpu core, it supports some v8_4A features, but
> some mandatory features are not implemented. Revert to ARMv8.2 that
> all mandatory features are supported.
>
Thanks, I've put this in.
I've modified the ChangeLog entry slightly - we normally use 'revert' in
the specific sense of completely removing an existing patch.
Also, when sending patches, please do not send ChangeLog entries as part
of the patch file. Because the file is always updated at the head, the
patch hunk is rarely going to apply cleanly. Instead, include the
ChangeLog text as part of your email description; that way we can then
paste it directly into the ChangeLog file itself and simply correct the
date.
R.
> ---
> gcc/ChangeLog | 5 +++++
> gcc/config/aarch64/aarch64-cores.def | 6 +++---
> 2 files changed, 8 insertions(+), 3 deletions(-)
>
> diff --git a/gcc/ChangeLog b/gcc/ChangeLog
> index e9f5baa6557c..842876b0ae90 100644
> --- a/gcc/ChangeLog
> +++ b/gcc/ChangeLog
> @@ -1,3 +1,8 @@
> +2018-12-19 Shaokun Zhang <zhangshaokun@hisilicon.com>
> +
> + * config/aarch64/aarch64-cores.def (tsv110) : Revert support for ARMv8.2
> + in tsv110.
> +
> 2018-12-18 Vladimir Makarov <vmakarov@redhat.com>
>
> PR rtl-optimization/87759
> diff --git a/gcc/config/aarch64/aarch64-cores.def b/gcc/config/aarch64/aarch64-cores.def
> index 74be5dbf2595..20f4924e084d 100644
> --- a/gcc/config/aarch64/aarch64-cores.def
> +++ b/gcc/config/aarch64/aarch64-cores.def
> @@ -96,10 +96,10 @@ AARCH64_CORE("cortex-a75", cortexa75, cortexa57, 8_2A, AARCH64_FL_FOR_ARCH8_2
> AARCH64_CORE("cortex-a76", cortexa76, cortexa57, 8_2A, AARCH64_FL_FOR_ARCH8_2 | AARCH64_FL_F16 | AARCH64_FL_RCPC | AARCH64_FL_DOTPROD, cortexa72, 0x41, 0xd0b, -1)
> AARCH64_CORE("ares", ares, cortexa57, 8_2A, AARCH64_FL_FOR_ARCH8_2 | AARCH64_FL_F16 | AARCH64_FL_RCPC | AARCH64_FL_DOTPROD | AARCH64_FL_PROFILE, cortexa72, 0x41, 0xd0c, -1)
>
> -/* ARMv8.4-A Architecture Processors. */
> -
> /* HiSilicon ('H') cores. */
> -AARCH64_CORE("tsv110", tsv110, cortexa57, 8_4A, AARCH64_FL_FOR_ARCH8_4 | AARCH64_FL_CRYPTO | AARCH64_FL_F16 | AARCH64_FL_AES | AARCH64_FL_SHA2, tsv110, 0x48, 0xd01, -1)
> +AARCH64_CORE("tsv110", tsv110, cortexa57, 8_2A, AARCH64_FL_FOR_ARCH8_2 | AARCH64_FL_CRYPTO | AARCH64_FL_F16 | AARCH64_FL_AES | AARCH64_FL_SHA2, tsv110, 0x48, 0xd01, -1)
> +
> +/* ARMv8.4-A Architecture Processors. */
>
> /* Qualcomm ('Q') cores. */
> AARCH64_CORE("saphira", saphira, saphira, 8_4A, AARCH64_FL_FOR_ARCH8_4 | AARCH64_FL_CRYPTO | AARCH64_FL_RCPC, saphira, 0x51, 0xC01, -1)
>
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] [aarch64] Revert support for ARMv8.2 in tsv110
2018-12-19 10:12 ` Richard Earnshaw (lists)
@ 2018-12-20 1:12 ` Zhangshaokun
0 siblings, 0 replies; 3+ messages in thread
From: Zhangshaokun @ 2018-12-20 1:12 UTC (permalink / raw)
To: Richard Earnshaw (lists), gcc-patches
Cc: james.greenhalgh, marcus.shawcroft, richard.sandiford,
kyrylo.tkachov, nickc, wuyuan5
Hi Richard,
On 2018/12/19 18:12, Richard Earnshaw (lists) wrote:
> On 19/12/2018 03:11, Shaokun Zhang wrote:
>> For HiSilicon's tsv110 cpu core, it supports some v8_4A features, but
>> some mandatory features are not implemented. Revert to ARMv8.2 that
>> all mandatory features are supported.
>>
>
> Thanks, I've put this in.
>
Thanks.
> I've modified the ChangeLog entry slightly - we normally use 'revert' in
> the specific sense of completely removing an existing patch.
>
I have checked the modified ChangeLog that is precise. Thanks for more explanation
about 'revert', got it.
> Also, when sending patches, please do not send ChangeLog entries as part
> of the patch file. Because the file is always updated at the head, the
> patch hunk is rarely going to apply cleanly. Instead, include the
> ChangeLog text as part of your email description; that way we can then
Surely, I will follow it. At the beginning, I also had the doubt that every
one would update the ChangeLog when he upstreamed the patch, how to apply
the patch directly if the ChangeLog file is conflicted. I have understood
it when you given the detailed description.
Thanks,
Shaokun
> paste it directly into the ChangeLog file itself and simply correct the
> date.
>
> R.
>
>> ---
>> gcc/ChangeLog | 5 +++++
>> gcc/config/aarch64/aarch64-cores.def | 6 +++---
>> 2 files changed, 8 insertions(+), 3 deletions(-)
>>
>> diff --git a/gcc/ChangeLog b/gcc/ChangeLog
>> index e9f5baa6557c..842876b0ae90 100644
>> --- a/gcc/ChangeLog
>> +++ b/gcc/ChangeLog
>> @@ -1,3 +1,8 @@
>> +2018-12-19 Shaokun Zhang <zhangshaokun@hisilicon.com>
>> +
>> + * config/aarch64/aarch64-cores.def (tsv110) : Revert support for ARMv8.2
>> + in tsv110.
>> +
>> 2018-12-18 Vladimir Makarov <vmakarov@redhat.com>
>>
>> PR rtl-optimization/87759
>> diff --git a/gcc/config/aarch64/aarch64-cores.def b/gcc/config/aarch64/aarch64-cores.def
>> index 74be5dbf2595..20f4924e084d 100644
>> --- a/gcc/config/aarch64/aarch64-cores.def
>> +++ b/gcc/config/aarch64/aarch64-cores.def
>> @@ -96,10 +96,10 @@ AARCH64_CORE("cortex-a75", cortexa75, cortexa57, 8_2A, AARCH64_FL_FOR_ARCH8_2
>> AARCH64_CORE("cortex-a76", cortexa76, cortexa57, 8_2A, AARCH64_FL_FOR_ARCH8_2 | AARCH64_FL_F16 | AARCH64_FL_RCPC | AARCH64_FL_DOTPROD, cortexa72, 0x41, 0xd0b, -1)
>> AARCH64_CORE("ares", ares, cortexa57, 8_2A, AARCH64_FL_FOR_ARCH8_2 | AARCH64_FL_F16 | AARCH64_FL_RCPC | AARCH64_FL_DOTPROD | AARCH64_FL_PROFILE, cortexa72, 0x41, 0xd0c, -1)
>>
>> -/* ARMv8.4-A Architecture Processors. */
>> -
>> /* HiSilicon ('H') cores. */
>> -AARCH64_CORE("tsv110", tsv110, cortexa57, 8_4A, AARCH64_FL_FOR_ARCH8_4 | AARCH64_FL_CRYPTO | AARCH64_FL_F16 | AARCH64_FL_AES | AARCH64_FL_SHA2, tsv110, 0x48, 0xd01, -1)
>> +AARCH64_CORE("tsv110", tsv110, cortexa57, 8_2A, AARCH64_FL_FOR_ARCH8_2 | AARCH64_FL_CRYPTO | AARCH64_FL_F16 | AARCH64_FL_AES | AARCH64_FL_SHA2, tsv110, 0x48, 0xd01, -1)
>> +
>> +/* ARMv8.4-A Architecture Processors. */
>>
>> /* Qualcomm ('Q') cores. */
>> AARCH64_CORE("saphira", saphira, saphira, 8_4A, AARCH64_FL_FOR_ARCH8_4 | AARCH64_FL_CRYPTO | AARCH64_FL_RCPC, saphira, 0x51, 0xC01, -1)
>>
>
>
> .
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-12-20 1:12 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-19 3:12 [PATCH] [aarch64] Revert support for ARMv8.2 in tsv110 Shaokun Zhang
2018-12-19 10:12 ` Richard Earnshaw (lists)
2018-12-20 1:12 ` Zhangshaokun
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).