[v2] config/arm: fix Neoverse N2 march flag
Checks
Commit Message
From: Pavan Nikhilesh <pbhagavatula@marvell.com>
The -march flag of Neoverse N2 should be removed with the introduction
of -mcpu='neoverse-n2' as they are incompatible.
Fixes: ea85ed1f6882 ("config/arm: increase nodes and cores for Neoverse N2")
Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
Tested-by: Srikanth Yalavarthi <syalavarthi@marvell.com>
---
v2 Changes:
- Remove adding armv9-a to march as it is incompatible with older GCC.
config/arm/meson.build | 1 -
1 file changed, 1 deletion(-)
--
2.39.1
Comments
> -----Original Message-----
> From: pbhagavatula@marvell.com <pbhagavatula@marvell.com>
> Sent: Friday, June 9, 2023 3:35 AM
> To: jerinj@marvell.com; Ruifeng Wang <Ruifeng.Wang@arm.com>; Bruce Richardson
> <bruce.richardson@intel.com>
> Cc: dev@dpdk.org; Pavan Nikhilesh <pbhagavatula@marvell.com>; Srikanth Yalavarthi
> <syalavarthi@marvell.com>
> Subject: [PATCH v2] config/arm: fix Neoverse N2 march flag
>
> From: Pavan Nikhilesh <pbhagavatula@marvell.com>
>
> The -march flag of Neoverse N2 should be removed with the introduction of -mcpu='neoverse-
> n2' as they are incompatible.
>
> Fixes: ea85ed1f6882 ("config/arm: increase nodes and cores for Neoverse N2")
>
> Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
> Tested-by: Srikanth Yalavarthi <syalavarthi@marvell.com>
> ---
> v2 Changes:
> - Remove adding armv9-a to march as it is incompatible with older GCC.
>
> config/arm/meson.build | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/config/arm/meson.build b/config/arm/meson.build index 43f6a551a2..ae2226988e
> 100644
> --- a/config/arm/meson.build
> +++ b/config/arm/meson.build
> @@ -89,7 +89,6 @@ part_number_config_arm = {
> 'march': 'armv8.4-a',
> },
> '0xd49': {
> - 'march': 'armv8.5-a',
> 'march_features': ['sve2'],
> 'compiler_options': ['-mcpu=neoverse-n2'],
> 'flags': [
> --
> 2.39.1
Acked-by: Ruifeng Wang <ruifeng.wang@arm.com>
> > From: Pavan Nikhilesh <pbhagavatula@marvell.com>
> >
> > The -march flag of Neoverse N2 should be removed with the introduction of -mcpu='neoverse-
> > n2' as they are incompatible.
> >
> > Fixes: ea85ed1f6882 ("config/arm: increase nodes and cores for Neoverse N2")
> >
> > Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
> > Tested-by: Srikanth Yalavarthi <syalavarthi@marvell.com>
>
> Acked-by: Ruifeng Wang <ruifeng.wang@arm.com>
Applied, thanks.
@@ -89,7 +89,6 @@ part_number_config_arm = {
'march': 'armv8.4-a',
},
'0xd49': {
- 'march': 'armv8.5-a',
'march_features': ['sve2'],
'compiler_options': ['-mcpu=neoverse-n2'],
'flags': [