回复: [edk2-devel] [PATCH v2 00/13] BaseTools,CryptoPkg,MdePkg,OvmfPkg: Delete CLANG35,CLANG38,GCC48,GCC49, rename GCC5 to GCC, update CLANGDWARF, delete VS 2008-2013, EBC

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

Rebecca:


> -----邮件原件-----
> 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Rebecca Cran
> 发送时间: 2023年3月29日 1:31
> 收件人: devel@edk2.groups.io; Oliver Smith-Denny <osd@smith-denny.com>;
> Guomin Jiang <guomin.jiang@intel.com>; Xiaoyu Lu <xiaoyu1.lu@intel.com>;
> Jian J Wang <jian.j.wang@intel.com>; Jiewen Yao <jiewen.yao@intel.com>;
> Ard Biesheuvel <ardb+tianocore@kernel.org>; Jordan Justen
> <jordan.l.justen@intel.com>; Gerd Hoffmann <kraxel@redhat.com>; Bob Feng
> <bob.c.feng@intel.com>; Liming Gao <gaoliming@byosoft.com.cn>; Andrew
> Fish <afish@apple.com>; Leif Lindholm <quic_llindhol@quicinc.com>;
> Michael D Kinney <michael.d.kinney@intel.com>
> 抄送: Rebecca Cran <rebecca@bsdio.com>
> 主题: [edk2-devel] [PATCH v2 00/13] BaseTools,CryptoPkg,MdePkg,OvmfPkg:
> Delete CLANG35,CLANG38,GCC48,GCC49, rename GCC5 to GCC, update
> CLANGDWARF, delete VS 2008-2013, EBC
> 
> 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 GCC48 and GCC49.

GCC49 is one GCC tool chain without LTO enable option. GCC5 is another GCC tool chain with LTO enable option.

They have the different usage. I suggest to keep GCC49 and GCC5 both, and also keep their name as is. 

Thanks
Liming
> 
> - 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.
> 
> - Remove unused CYGWIN_ definitions.
> 
> - Rename GCC5 to GCC.
> 
> Personal GitHub PR: https://github.com/tianocore/edk2/pull/4158
> GitHub branch: https://github.com/bcran/edk2/tree/clangdwarf
> 
> Note: CI will fail due to changes that are needed in the edk2-basetools
> and containers repositories.
> 
> Changes from v1 to v2:
> 
> - Added a commit to delete GCC48 and GCC49, rename GCC5 to GCC and
>   update the flags for other toolchains to work with the new GCC
>   definitions.
> 
> - Bumped VERSION from 2.00 to 3.00 to inform users that they should
>   updated their Conf/tools_def.txt.
> 
> Rebecca Cran (13):
>   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
>   OvmfPkg: Use Xcode5 version of CpuExceptionHandlerLib for
> CLANGDWARF
>   Delete GCC48 and GCC49 toolchains and rename GCC5 to GCC
>   BaseTools: Remove unused CYGWIN_ defines from tools_def.txt.template
>   Bump tools_def.txt VERSION to 3.0
> 
>  UnitTestFrameworkPkg/UnitTestFrameworkPkgHost.dsc.inc
> |    4 +-
>  IntelFsp2Pkg/Tools/Tests/QemuFspPkg.dsc
> |   12 +-
>  OvmfPkg/OvmfPkgX64.dsc
> |    2 +-
>  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
> |    5 +-
>  CryptoPkg/Library/OpensslLib/OpensslLibAccel.inf                    |
> 5 +-
>  CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf                   |
> 5 +-
>  CryptoPkg/Library/OpensslLib/OpensslLibFull.inf                     |
> 5 +-
>  CryptoPkg/Library/OpensslLib/OpensslLibFullAccel.inf                |
> 5 +-
>  EmulatorPkg/Unix/Host/Host.inf
> |    6 +-
>  MdePkg/Include/Ia32/ProcessorBind.h
> |    8 +-
>  MdePkg/Include/X64/ProcessorBind.h
> |    8 +-
>  OvmfPkg/Library/PlatformInitLib/MemDetect.c
> |    4 +-
>  OvmfPkg/Library/X86QemuLoadImageLib/X86QemuLoadImageLib.c
> |    5 -
>  .azurepipelines/Ubuntu-GCC5.yml
> |    4 +-
>  .azurepipelines/templates/platform-build-run-steps.yml              |
> 2 +-
>  .azurepipelines/templates/pr-gate-build-job.yml                     |
> 2 +-
>  .azurepipelines/templates/pr-gate-steps.yml
> |    2 +-
>  ArmPlatformPkg/Scripts/Makefile
> |    2 +-
>  ArmVirtPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
> |    4 +-
>  ArmVirtPkg/PlatformCI/ReadMe.md
> |    2 +-
>  BaseTools/Conf/tools_def.template
> | 2528 ++++----------------
>  BaseTools/Edk2ToolsBuild.py
> |    2 +-
>  BaseTools/Plugin/HostBasedUnitTestRunner/HostBasedUnitTestRunner.py |
> 2 +-
>  BaseTools/Plugin/LinuxGcc5ToolChain/LinuxGcc5ToolChain.py           |
> 62 +-
>  BaseTools/Plugin/LinuxGcc5ToolChain/LinuxGcc5ToolChain_plug_in.yaml |
> 4 +-
>  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 +-
>  DynamicTablesPkg/Readme.md
> |   10 +-
>  EmulatorPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
> |    4 +-
>  EmulatorPkg/PlatformCI/ReadMe.md
> |    2 +-
>  EmulatorPkg/Readme.md
> |    8 +-
>  EmulatorPkg/build.sh
> |   14 +-
>  OvmfPkg/CloudHv/README
> |    2 +-
>  OvmfPkg/IntelTdx/README
> |    4 +-
>  OvmfPkg/PlatformCI/.azurepipelines/Ubuntu-GCC5.yml
> |    4 +-
>  OvmfPkg/PlatformCI/ReadMe.md
> |    2 +-
>  OvmfPkg/README
> |    4 +-
>  OvmfPkg/build.sh
> |   12 +-
>  PrmPkg/Readme.md
> |    6 +-
>  ReadMe.rst
> |  102 +-
>  UnitTestFrameworkPkg/ReadMe.md
> |    8 +-
>  edksetup.bat
> |    6 +-
>  51 files changed, 592 insertions(+), 2471 deletions(-)
> 
> --
> 2.34.1
> 
> 
> 
> 
>