回复: [edk2-devel] [PATCH 0/9] BaseTools,CryptoPkg,MdePkg,OvmfPkg: Delete CLANG35,CLANG38 and update CLANGDWARF, delete VS 2008-2013, EBC

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

Rebecca:
  Can you submit BZ for this change? This change belongs to the feature. I would like to add it into stable tag feature planning. 

Thanks
Liming
> -----邮件原件-----
> 发件人: Oliver Smith-Denny <osd@smith-denny.com>
> 发送时间: 2023年3月23日 0:37
> 收件人: devel@edk2.groups.io; rebecca@bsdio.com; Michael D Kinney
> <michael.d.kinney@intel.com>; Liming Gao <gaoliming@byosoft.com.cn>;
> Zhiguang Liu <zhiguang.liu@intel.com>; Bob Feng <bob.c.feng@intel.com>;
> Yuwei Chen <yuwei.chen@intel.com>; Andrew Fish <afish@apple.com>; Leif
> Lindholm <quic_llindhol@quicinc.com>; Ard Biesheuvel
> <ardb+tianocore@kernel.org>; Jordan Justen <jordan.l.justen@intel.com>;
> Gerd Hoffmann <kraxel@redhat.com>
> 主题: Re: [edk2-devel] [PATCH 0/9] BaseTools,CryptoPkg,MdePkg,OvmfPkg:
> Delete CLANG35,CLANG38 and update CLANGDWARF, delete VS 2008-2013,
> EBC
> 
> Thanks for the cleanup!
> 
> Reviewed-by: Oliver Smith-Denny <osd@smith-denny.com>
> 
> On 3/21/2023 6:30 PM, Rebecca Cran wrote:
> > Update the toolchain definitions:
> >
> > - Delete the CLANG35 and CLANG38 toolchains, and replace CLANG38 with
> >    CLANGDWARF, updating it to support ARM and AARCH64 in addition to
> X64
> > and IA32.
> >
> > - Remove VS2008, VS2010, VS2012 and VS2013.
> >
> > - Remove EBC compiler definitions. Full removal of EBC support from the
> >    various packages etc. will be done in a follow-up patch series.
> >
> > - Remove unused IPHONE_TOOLS and SOURCERY_CYGWIN_TOOLS
> definitions.
> >
> > Personal GitHub PR: https://github.com/tianocore/edk2/pull/4158
> > GitHub branch: https://github.com/bcran/edk2/tree/clangdwarf
> >
> > Rebecca Cran (9):
> >    OvmfPkg: Replace static struct initialization with ZeroMem call
> >    CryptoPkg: Add CLANGDWARF and remove CLANG35 and CLANG38
> compiler
> >      flags
> >    BaseTools: Update CLANGDWARF toolchain and remove CLANG35 and
> CLANG38
> >    BaseTools: Remove VS2008, 2010, 2012 and 2013 toolchain definitions
> >    BaseTools: Remove VS2008-VS2013 remnants
> >    MdePkg: Remove VS2008-VS2013 remnants
> >    edksetup.bat: Remove VS2008-VS2013 remnants
> >    BaseTools: Remove unused IPHONE_TOOLS and
> SOURCERY_CYGWIN_TOOLS defs
> >    BaseTools: Remove EBC (EFI Byte Code) compiler definitions
> >
> >   CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf             |
> 3 +-
> >   CryptoPkg/Library/BaseCryptLib/PeiCryptLib.inf              |    3
> +-
> >   CryptoPkg/Library/BaseCryptLib/RuntimeCryptLib.inf          |
> 3 +-
> >   CryptoPkg/Library/BaseCryptLib/SecCryptLib.inf              |
> 3 +-
> >   CryptoPkg/Library/BaseCryptLib/SmmCryptLib.inf              |
> 3 +-
> >   CryptoPkg/Library/BaseCryptLib/UnitTestHostBaseCryptLib.inf |    3
> +-
> >   CryptoPkg/Library/OpensslLib/OpensslLib.inf                 |
> 3 +-
> >   CryptoPkg/Library/OpensslLib/OpensslLibAccel.inf            |    3
> +-
> >   CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf           |    3
> +-
> >   CryptoPkg/Library/OpensslLib/OpensslLibFull.inf             |    3
> +-
> >   CryptoPkg/Library/OpensslLib/OpensslLibFullAccel.inf        |    3
> +-
> >   MdePkg/Include/Ia32/ProcessorBind.h                         |
> 8 +-
> >   MdePkg/Include/X64/ProcessorBind.h                          |
> 8 +-
> >   OvmfPkg/Library/PlatformInitLib/MemDetect.c                 |
> 4 +-
> >   BaseTools/Conf/tools_def.template                           |
> 1572 +++-----------------
> >   BaseTools/Scripts/SetVisualStudio.bat                       |
> 22 +-
> >   BaseTools/Scripts/ShowEnvironment.bat                       |
> 44 -
> >   BaseTools/get_vsvars.bat                                    |
> 13 -
> >   BaseTools/set_vsprefix_envs.bat                             |
> 64 -
> >   BaseTools/toolsetup.bat                                     |
> 24 +-
> >   edksetup.bat
> |    6 +-
> >   21 files changed, 192 insertions(+), 1606 deletions(-)
> >