RE: [PATCH] arm: Add missing part number for Neoverse V1

2020-10-02 Thread Kyrylo Tkachov via Gcc-patches 


> -Original Message-
> From: Alex Coplan 
> Sent: 02 October 2020 15:41
> To: gcc-patches@gcc.gnu.org
> Cc: ni...@redhat.com; Richard Earnshaw ;
> Ramana Radhakrishnan ; Kyrylo
> Tkachov 
> Subject: [PATCH] arm: Add missing part number for Neoverse V1
> 
> This patch adds vendor and part numbers which were missing from the
> initial entry for Neoverse V1 in AArch32 GCC.
> 
> OK for trunk and backports to GCC 10 and 9?

Ok.
Thanks,
Kyrill

> 
> I believe GCC 8 handles these differently so that will need fixing
> separately.
> 
> Thanks,
> Alex
> 
> ---
> 
> gcc/ChangeLog:
> 
>   * config/arm/arm-cpus.in (neoverse-v1): Add missing vendor and
>   part numbers.

[PATCH] arm: Add missing part number for Neoverse V1

2020-10-02 Thread Alex Coplan via Gcc-patches 
This patch adds vendor and part numbers which were missing from the
initial entry for Neoverse V1 in AArch32 GCC.

OK for trunk and backports to GCC 10 and 9?

I believe GCC 8 handles these differently so that will need fixing
separately.

Thanks,
Alex

---

gcc/ChangeLog:

* config/arm/arm-cpus.in (neoverse-v1): Add missing vendor and
part numbers.
diff --git a/gcc/config/arm/arm-cpus.in b/gcc/config/arm/arm-cpus.in
index 9abb59a00ba..27ce0001633 100644
--- a/gcc/config/arm/arm-cpus.in
+++ b/gcc/config/arm/arm-cpus.in
@@ -1519,6 +1519,8 @@ begin cpu neoverse-v1
   architecture armv8.4-a+fp16+bf16+i8mm
   option crypto add FP_ARMv8 CRYPTO
   costs cortex_a57
+  vendor 41
+  part 0xd40
 end cpu neoverse-v1
 
 # Armv8.5 A-profile Architecture Processors