From 8594c2073cdb1065e60f01e9b099918c5e839212 Mon Sep 17 00:00:00 2001 From: Ard Biesheuvel Date: Thu, 20 Dec 2018 20:16:53 +0100 Subject: [PATCH 1/1] ArmPkg/ArmLib ARM: set .fpu to let Clang 7 assemble ArmV7Support.S Clang 7 complains about the vmsr instruction in ArmV7Support.S, which is only available on cores that implement some flavour of VFP. So set the .fpu to NEON like we do in some other places. Acked-by: Laszlo Ersek Reviewed-by: Leif Lindholm Reviewed-by: Philippe Mathieu-Daude Signed-off-by: Ard Biesheuvel --- ArmPkg/Library/ArmLib/Arm/ArmV7Support.S | 2 ++ 1 file changed, 2 insertions(+) diff --git a/ArmPkg/Library/ArmLib/Arm/ArmV7Support.S b/ArmPkg/Library/ArmLib/Arm/ArmV7Support.S index 16c56f72e9..4925f6628e 100644 --- a/ArmPkg/Library/ArmLib/Arm/ArmV7Support.S +++ b/ArmPkg/Library/ArmLib/Arm/ArmV7Support.S @@ -262,6 +262,8 @@ ASM_FUNC(ArmEnableVFP) #ifndef __clang__ mcr p10,#0x7,r0,c8,c0,#0 #else + # Set the FPU model so Clang does not choke on the next instruction + .fpu neon vmsr fpexc, r0 #endif bx lr -- 2.39.2