Re: [Minios-devel] [UNIKRAFT PATCHv4 12/43] arch/arm: Check gcc version and set processor flags for arm32

[Simon Kuenzer <simon.kuenzer@xxxxxxxxx>]

Hey,

the discussion we had with patch 7/43 is going to have implications to 
this patch as well.
Thanks

Simon

On 06.07.2018 11:03, Wei Chen wrote:
> Similar to ARM64, the optimization processor flags have the minimum
> gcc version requirement. So we still need to check the GCC version
> for arm32. As we have already defined a makefile function to do such
> task, so we can use this function for arm32 too.
>
> Signed-off-by: Wei Chen <Wei.Chen@xxxxxxx>
> ---
>   arch/arm/arm/Makefile.uk | 13 ++++++++-----
>   1 file changed, 8 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm/arm/Makefile.uk b/arch/arm/arm/Makefile.uk
> index a84f2b8..a187946 100644
> --- a/arch/arm/arm/Makefile.uk
> +++ b/arch/arm/arm/Makefile.uk
> @@ -5,12 +5,15 @@ CFLAGS   += -marm -fms-extensions
>   CXXFLAGS += -D__ARM_32__
>   CXXFLAGS += -marm -fms-extensions
>   
> -#-march=armv7-a
> +# Set GCC flags for MARCH_ARM32_A20NEON. GCC supports -mcpu=cortex-a7 from 4.7
> +ifeq ($(CONFIG_MARCH_ARM32_A20NEON),y)
> +$(eval $(call set_flags_if_gcc_version_ge,4,7,-mcpu=cortex-a7 -mtune=cortex-a7 
> -mfpu=vfpv4-d16 -mfpu=neon-vfpv4 -funsafe-math-optimizations))
> +endif
>   
> -CFLAGS-$(CONFIG_MARCH_ARM32_A20NEON)    += -mcpu=cortex-a7 -mtune=cortex-a7 -mfpu=vfpv4-d16 -mfpu=neon-vfpv4 -funsafe-math-optimizations
> -CXXFLAGS-$(CONFIG_MARCH_ARM32_A20NEON)  += -mcpu=cortex-a7 -mtune=cortex-a7 
> -mfpu=vfpv4-d16 -mfpu=neon-vfpv4 -funsafe-math-optimizations
> -CFLAGS-$(CONFIG_MARCH_ARM32_CORTEXA7)   += -mcpu=cortex-a7 -mtune=cortex-a7
> -CXXFLAGS-$(CONFIG_MARCH_ARM32_CORTEXA7) += -mcpu=cortex-a7 -mtune=cortex-a
> +# Set GCC flags for MARCH_ARM32_CORTEXA7. GCC supports -mcpu=cortex-a7 from 4.7
> +ifeq ($(CONFIG_MARCH_ARM32_CORTEXA7),y)
> +$(eval $(call set_flags_for_valid_gcc,4,7,-mcpu=cortex-a7 -mtune=cortex-a7))
> +endif
>   
>   $(eval $(call addlib,libarmmath))
>   LIBARMMATH_SRCS-$(CONFIG_ARCH_ARM_32) += $(CONFIG_UK_BASE)/arch/arm/divsi3.S
_______________________________________________
Minios-devel mailing list
Minios-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/minios-devel