Re: [edk2-devel] [PATCH 3/3] Basetools: turn off gcc12 warning

2022-03-29 Thread Andrew Fish via groups.io
> On Mar 29, 2022, at 2:38 PM, Andrew Fish via groups.io > wrote: > > > >> On Mar 29, 2022, at 11:14 AM, gr...@grantlmul.xyz >> wrote: >> >> While I can't test this myself, the following does appear to result in a >> successful build using the same clang versi

Re: [edk2-devel] [PATCH 3/3] Basetools: turn off gcc12 warning

2022-03-29 Thread Andrew Fish via groups.io
> On Mar 29, 2022, at 11:14 AM, gr...@grantlmul.xyz wrote: > > While I can't test this myself, the following does appear to result in a > successful build using the same clang version: > > > diff --git a/BaseTools/Source/C/DevicePath/GNUmakefile > b/BaseTools/Source/C/DevicePath/GNUmakefile

Re: [edk2-devel] [PATCH 3/3] Basetools: turn off gcc12 warning

2022-03-29 Thread grant
While I can't test this myself, the following does appear to result in a successful build using the same clang version: diff --git a/BaseTools/Source/C/DevicePath/GNUmakefile b/BaseTools/Source/C/DevicePath/GNUmakefile index b05d2bddfa..81aa35aa31 100644 --- a/BaseTools/Source/C/DevicePath/GNU

Re: [edk2-devel] [PATCH 3/3] Basetools: turn off gcc12 warning

2022-03-28 Thread Bob Feng
Steffen ; Feng, Bob C Subject: Re: [edk2-devel] [PATCH 3/3] Basetools: turn off gcc12 warning This breaks building BaseTools with clang 13.1.6 on macOS: /Applications/Xcode.app/Contents/Developer/usr/bin/make -C DevicePath gcc  -c  -I .. -I ../Include/Common -I ../Include/ -I ../Include

Re: [edk2-devel] [PATCH 3/3] Basetools: turn off gcc12 warning

2022-03-28 Thread Rebecca Cran
This breaks building BaseTools with clang 13.1.6 on macOS: /Applications/Xcode.app/Contents/Developer/usr/bin/make -C DevicePath gcc  -c  -I .. -I ../Include/Common -I ../Include/ -I ../Include/IndustryStandard -I ../Common/ -I .. -I . -I ../Include/AArch64/ -MD -fshort-wchar -fno-strict-alias

[edk2-devel] [PATCH 3/3] Basetools: turn off gcc12 warning

2022-03-24 Thread Gerd Hoffmann
In function ‘SetDevicePathEndNode’, inlined from ‘FileDevicePath’ at DevicePathUtilities.c:857:5: DevicePathUtilities.c:321:3: error: writing 4 bytes into a region of size 1 [-Werror=stringop-overflow=] 321 | memcpy (Node, &mUefiDevicePathLibEndDevicePath, sizeof (mUefiDevicePathLibEndDev