回复: [edk2-devel] [PATCH] BaseTools/tools_def: Disable overzealous unused variable warning on Clang

["gaoliming" <gaoliming@byosoft.com.cn>]

Ard:
  I am OK. 

Thanks
Liming
> -----邮件原件-----
> 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Ard
> Biesheuvel
> 发送时间: 2023年5月12日 14:40
> 收件人: devel@edk2.groups.io; kraxel@redhat.com
> 抄送: Rebecca Cran <rebecca@bsdio.com>; Liming Gao
> <gaoliming@byosoft.com.cn>
> 主题: Re: [edk2-devel] [PATCH] BaseTools/tools_def: Disable overzealous
> unused variable warning on Clang
> 
> Liming:
> 
> I'd like to merge this one for the stable tag - any objections?
> 
> 
> On Thu, 11 May 2023 at 06:55, Gerd Hoffmann <kraxel@redhat.com> wrote:
> >
> > On Wed, May 10, 2023 at 04:50:08PM +0200, Ard Biesheuvel wrote:
> > > The warnings Clang emits when enabling
> -Wunneeded-internal-declaration
> > > (which is part of -Wall) are generating false positives for variables
> > > whose size gets taken but are not referenced beyond that.
> > >
> > > This may happen legitimately in debug code, so let's disable this
> > > warning for Clang, rather than tiptoe around it in the code base.
> > >
> > > This fixes the RELEASE build for OVMF, which triggers this issue in the
> > > newly added VirtioSerialDxe driver.
> > >
> > > Cc: Rebecca Cran <rebecca@bsdio.com>
> > > Cc: Gerd Hoffmann <kraxel@redhat.com>
> > > Cc: Liming Gao <gaoliming@byosoft.com.cn>
> > > Signed-off-by: Ard Biesheuvel <ardb@kernel.org>
> >
> > Acked-by: Gerd Hoffmann <kraxel@redhat.com>
> >
> >
> >
> >
> >
> >
> 
> 
> 
>