Re: [edk2] [PATCH 3/3] BaseTools GCC/ARM: add -fno-builtin to CC flags

2016-08-05 Thread Ard Biesheuvel
On 4 August 2016 at 18:01, Michael Zimmermann wrote: > not directly related but could we add nostdinc too? At least for my platform > that works perfectly and it prevents you from accidentally including any > libc/libgcc/whatever headers from your toolchain.(nostdlib is

Re: [edk2] [PATCH 3/3] BaseTools GCC/ARM: add -fno-builtin to CC flags

2016-08-05 Thread Ard Biesheuvel
On 5 August 2016 at 16:26, Gao, Liming wrote: > Reviewed-by: Liming Gao > Thanks all Pushed as 59ceaa0a871d ArmPkg/ArmSoftFloatLib: disable LTO build for GCC f8c51389c6db ArmPkg/CompilerIntrinsicsLib: make the default memset() weak 0667e985270b

Re: [edk2] [PATCH 3/3] BaseTools GCC/ARM: add -fno-builtin to CC flags

2016-08-04 Thread Michael Zimmermann
not directly related but could we add nostdinc too? At least for my platform that works perfectly and it prevents you from accidentally including any libc/libgcc/whatever headers from your toolchain.(nostdlib is already enabled) Thanks Michael On Thu, Aug 4, 2016 at 4:42 PM, Ard Biesheuvel

[edk2] [PATCH 3/3] BaseTools GCC/ARM: add -fno-builtin to CC flags

2016-08-04 Thread Ard Biesheuvel
Avoid build errors when including OpensslLib, which may throw undefined reference errors for builtin functions if -fno-builtin is not specified (and it is already set for IA32, X64 and AARCH64) So set it for ARM as well. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard