Re: [PATCH edk2-platforms 2/5] Platform/StandaloneMm: Remove '-march=armv8-a+nofp' flag

["Thomas Abraham" <thomas.abraham@arm.com>]

On 25/01/2023 10:27, Pierre.Gondois@arm.com wrote:
> From: Pierre Gondois <pierre.gondois@arm.com>
> 
> The '-march=armv8-a+nofp' flag used for PlatformStandaloneMm.dsc
> was removed in the previous patch. The same flag was probably copied
> from this first configuration in:
> commit bd5071cfbde01 ("StMMRpmb: Add support for building StandaloneMm
> image for OP-TEE")
> 
> As the flag is removed for PlatformStandaloneMm.dsc in the previous
> patch, also remove the flag for PlatformStandaloneMmRpmb.dsc.
> 
> CC: Ilias Apalodimas <ilias.apalodimas@linaro.org>
> CC: Omkar Anand Kulkarni <omkar.kulkarni@arm.com>
> CC: Pranav Madhu <pranav.madhu@arm.com>
> CC: Sayanta Pattanayak <Sayanta.Pattanayak@arm.com>
> Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
> ---
>   .../PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.dsc        | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Platform/StandaloneMm/PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.dsc b/Platform/StandaloneMm/PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.dsc
> index 653029c52fdb..611d76e62117 100644
> --- a/Platform/StandaloneMm/PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.dsc
> +++ b/Platform/StandaloneMm/PlatformStandaloneMmPkg/PlatformStandaloneMmRpmb.dsc
> @@ -165,7 +165,7 @@ [Components.common]
>   #
>   ###################################################################################################
>   [BuildOptions.AARCH64]
> -GCC:*_*_*_DLINK_FLAGS = -z common-page-size=0x1000 -march=armv8-a+nofp
> +GCC:*_*_*_DLINK_FLAGS = -z common-page-size=0x1000
>   GCC:*_*_*_CC_FLAGS = -mstrict-align
>   
>   [BuildOptions.ARM]


Reviewed-by: Thomas Abraham <thomas.abraham@arm.com>