From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mout.perfora.net (mout.perfora.net [74.208.4.197]) by mx.groups.io with SMTP id smtpd.web10.48544.1679503053762792095 for ; Wed, 22 Mar 2023 09:37:34 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=none, err=permanent DNS error (domain: smith-denny.com, ip: 74.208.4.197, mailfrom: osd@smith-denny.com) Received: from [10.137.194.171] ([131.107.8.107]) by mrelay.perfora.net (mreueus003 [74.208.5.2]) with ESMTPSA (Nemesis) id 0LqjVU-1q9QGA3WwU-00eKUi; Wed, 22 Mar 2023 17:37:13 +0100 Message-ID: Date: Wed, 22 Mar 2023 09:37:09 -0700 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.8.0 Subject: Re: [edk2-devel] [PATCH 0/9] BaseTools,CryptoPkg,MdePkg,OvmfPkg: Delete CLANG35,CLANG38 and update CLANGDWARF, delete VS 2008-2013, EBC To: devel@edk2.groups.io, rebecca@bsdio.com, Michael D Kinney , Liming Gao , Zhiguang Liu , Bob Feng , Yuwei Chen , Andrew Fish , Leif Lindholm , Ard Biesheuvel , Jordan Justen , Gerd Hoffmann References: <20230322013040.3322865-1-rebecca@bsdio.com> From: "Oliver Smith-Denny" In-Reply-To: <20230322013040.3322865-1-rebecca@bsdio.com> X-Provags-ID: V03:K1:ClGrMDEhNMxf5lFKWVAwVXkWStZk3cwe7XiHPURPVMFZ3z9uY3g CGoATC9qRJNV4A8aAMNAIvG6lJGhVeIWDiQ3T3xeLflR0PvrQWCtgNq4l3cBetFqI34cSi1 hrujRYo+ezfUOejl3ipRh5I9AbpmeyEcEsyR0VlkGWjaKxShaESx0ALQZAd+b9CPLkao9te du7uE2nfUgB29Tme+4WNA== X-Spam-Flag: NO UI-OutboundReport: notjunk:1;M01:P0:TVpuyg7pBxE=;vaHlcM0GjNWMu/j6Vd8QKlMWVZF ftYpEhD9SpBlEBrb7oY5CJ7F/R8aRKADaxoVntgoyA7vSbBRLyfMvRFTB0mFmtxSWCy77R3/E 0Dzgfk3lofrMk0d4zXKEPj7gkedmuJ2FIVxklse1Rz/lvlpmJavaG1IPnyIrWiJLz6W2pL/ns tGJq1ricld+bg1ug1GRuXmF6Tg6iD9eyAF4PJrGaRnVKOVjRgzzH48V01FjSBpo00tRK+JgYV 9dMtr1KAlzAcjMKtiUaMdO8imhy3ZqsmZ9haWW3K+c3aVVPVLnXqwSoQreHLt8qARzg/iniDN T615c6OO2hMh/n80AXFkQSTJwirMG3kYuYEQ087c2xrZQrwQ0bPZ8hmMaO1P1pBLysdJs7LtX gy3J91D1Xb4V3YpGPm3JzSuBk/2e1mJM09Lg+y+GyUkUf/YLQ6HR8VbNPINV3yYZ/fBzne4Cf kOFpTIejWvUgkKSH6t0pt1/lJFatOhuiRyycTEdag/1M6Y8hTk1d1D/BDvglWXnpOvw2l43jz alTxlVaWLU+Q93WmQiJWWUqf+N3SwS3NVe+iNeJacPS498IDtP/1b8xy8Vgc42Kjuevymy/b5 0VPECPrMEWkVxq6u61YHqNEC0razigho3Ph5mXVNNY4XbUTIekufwFhqg32Bp/PI8KL9ocjmm DtfI3EZv/hAFXMEwPeZjUhYW94K8TLHuB/O5zoxbMsh1ckY2wumSq6+GLcGNIN8= Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Thanks for the cleanup! Reviewed-by: Oliver Smith-Denny 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(-) >