On 03/07/18 07:43, Gao, Liming wrote: > Laszlo: > We just find this change causes XCODE5 build failure, because XCODE > compiler doesn't know these two options. So, could you provide the patch to > remove the change in BUILD_CFLAGS for MAC?
Yep, I was a bit concerned about that. Ultimately I assumed that - clang would either recognize these warnings options (because it does recognize most -W options of gcc), - or, similarly to gcc, clang would simply ignore -Wno-* flags that it does not recognize. Neither appears to be the case, then. I'll send a patch soon. Thanks, and sorry about the regression. Laszlo > ifeq ($(DARWIN),Darwin) > # assume clang or clang compatible flags on OS X > BUILD_CFLAGS = > else > > Thanks > Liming >> -----Original Message----- >> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Gao, >> Liming >> Sent: Monday, March 5, 2018 10:14 PM >> To: Laszlo Ersek <ler...@redhat.com>; edk2-devel-01 <edk2-devel@lists.01.org> >> Cc: Paolo Bonzini <pbonz...@redhat.com>; Cole Robinson >> <crobi...@redhat.com>; Ard Biesheuvel <ard.biesheu...@linaro.org> >> Subject: Re: [edk2] [PATCH 0/3] BaseTools: let the C-language build utils >> compile with gcc-8 >> >> Reviewed-by: Liming Gao <liming....@intel.com> >> >>> -----Original Message----- >>> From: Laszlo Ersek [mailto:ler...@redhat.com] >>> Sent: Saturday, March 03, 2018 2:09 AM >>> To: edk2-devel-01 <edk2-devel@lists.01.org> >>> Cc: Ard Biesheuvel <ard.biesheu...@linaro.org>; Cole Robinson >>> <crobi...@redhat.com>; Gao, Liming <liming....@intel.com>; Paolo Bonzini >>> <pbonz...@redhat.com>; Zhu, Yonghong <yonghong....@intel.com> >>> Subject: [PATCH 0/3] BaseTools: let the C-language build utils compile with >>> gcc-8 >>> >>> Repo: https://github.com/lersek/edk2.git >>> Branch: basetools_gcc8 >>> >>> Once these patches are applied to the build flags and the source code of >>> the build utilities themselves, OVMF builds fine with gcc-8, using the >>> GCC5 toolchain settings without any changes. >>> >>> Regression-tested with gcc-4.8 / x86_64. >>> >>> Cc: Ard Biesheuvel <ard.biesheu...@linaro.org> >>> Cc: Cole Robinson <crobi...@redhat.com> >>> Cc: Liming Gao <liming....@intel.com> >>> Cc: Paolo Bonzini <pbonz...@redhat.com> >>> Cc: Yonghong Zhu <yonghong....@intel.com> >>> >>> Thanks >>> Laszlo >>> >>> Laszlo Ersek (3): >>> BaseTools/header.makefile: add "-Wno-stringop-truncation" >>> BaseTools/header.makefile: add "-Wno-restrict" >>> BaseTools/GenVtf: silence false "stringop-overflow" warning with >>> memcpy() >>> >>> BaseTools/Source/C/Makefiles/header.makefile | 4 ++-- >>> BaseTools/Source/C/GenVtf/GenVtf.c | 4 ++-- >>> 2 files changed, 4 insertions(+), 4 deletions(-) >>> >>> -- >>> 2.14.1.3.gb7cf6e02401b >> >> _______________________________________________ >> edk2-devel mailing list >> edk2-devel@lists.01.org >> https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel