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

["Rebecca Cran" <rebecca@bsdio.com>]

I've created https://bugzilla.tianocore.org/show_bug.cgi?id=4384 .


-- 
Rebecca Cran


On 3/22/23 7:05 PM, gaoliming wrote:
> 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(-)
>>>
>