Re: [edk2-devel] MdePkg fails to build in RELEASE mode with clang 13 (CLANG38 toolset)

["Andrew Fish" <afish@apple.com>]

[-- Attachment #1: Type: text/plain, Size: 1623 bytes --]

Mike,

That sounds like the right fox to me. Disable the warning.
> On Feb 2, 2022, at 1:18 PM, Michael D Kinney <michael.d.kinney@intel.com> wrote:
> 
> 
> Hi Rebecca,
>  
> GCC added -Wno-unused-but-set-variable back in 2016 for RELEASE builds to address this same issue
>  
> https://github.com/tianocore/edk2/commit/20d00edf21d2f2144921622891d8b59a1553cd83
>  
> Then later added -Wno-unused-const-variable
>  
> https://github.com/tianocore/edk2/commit/8b6366f8758426fd6710646574b9a7e583e0fe0d
>  
> Mike
>  
> From: Rebecca Cran <quic_rcran@quicinc.com> 
> Sent: Wednesday, February 2, 2022 12:52 PM
> To: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com>; devel@edk2.groups.io
> Subject: MdePkg fails to build in RELEASE mode with clang 13 (CLANG38 toolset)
>  
> With the CLANG38 toolset (clang version 13.0.0) MdePkg fails to build in RELEASE mode because Status is only used by an assert.
>  
> edk2/MdePkg/Library/UefiMemoryAllocationLib/MemoryAllocationLib.c:141:15: error: variable 'Status' set but not used [-Werror,-Wunused-but-set-variable] 
>  EFI_STATUS  Status; 
>              ^ 
> edk2/MdePkg/Library/UefiMemoryAllocationLib/MemoryAllocationLib.c:347:15: error: variable 'Status' set but not used [-Werror,-Wunused-but-set-variable] 
>  EFI_STATUS  Status; 
>              ^ 
> edk2/MdePkg/Library/UefiMemoryAllocationLib/MemoryAllocationLib.c:818:15: error: variable 'Status' set but not used [-Werror,-Wunused-but-set-variable] 
>  EFI_STATUS  Status;
> 
> -- 
> Rebecca Cran
> 
> 

[-- Attachment #2: Type: text/html, Size: 42830 bytes --]