From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from wout2-smtp.messagingengine.com (wout2-smtp.messagingengine.com [64.147.123.25]) by mx.groups.io with SMTP id smtpd.web10.48674.1683204155778269645 for ; Thu, 04 May 2023 05:42:36 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="signature has expired" header.i=@bsdio.com header.s=fm1 header.b=uOOQwnM9; spf=pass (domain: bsdio.com, ip: 64.147.123.25, mailfrom: rebecca@bsdio.com) Received: from compute5.internal (compute5.nyi.internal [10.202.2.45]) by mailout.west.internal (Postfix) with ESMTP id BC711320094F; Thu, 4 May 2023 08:42:33 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute5.internal (MEProxy); Thu, 04 May 2023 08:42:35 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdio.com; h=cc :content-transfer-encoding:content-type:content-type:date:date :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:sender:subject:subject:to:to; s=fm1; t= 1683204153; x=1683290553; bh=V2Mw0IxPH/jNvlAgjNDYsCJYqLM7jJdLuOa C4mbqPPo=; b=uOOQwnM9pdoF7k5trkc9QHZQu3+wNfSmgxdtZ+hq7ea/Q35qZvH JpsBxmapeIsWVKf6GGj3zLtZYfTpEt8OqDMOclkKfuaePjI9ut5UK9eDbR2Wi9EU P3JmR/t5F4JFr2prODYk1qT8WzWegl7hkX2H0AWBYqsBy9Jl6km4Kiw+bZ/VyhO0 3xgBYpgjL8dKG0DN5f68mN3cFd5bkXec7oDaaqYIlcItpzabDhtg5V/ujILsWwg/ u2gG1CHZs7eMdFLJ+jjbgDFDaMH79TePAdc+wlGI7gU4NM0tTeb4XH/raQqk1axm dpF6cJp+5txP6HOjbFlMTW/r3wFFfZYB68A== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :content-type:date:date:feedback-id:feedback-id:from:from :in-reply-to:in-reply-to:message-id:mime-version:references :reply-to:sender:subject:subject:to:to:x-me-proxy:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm3; t=1683204153; x= 1683290553; bh=V2Mw0IxPH/jNvlAgjNDYsCJYqLM7jJdLuOaC4mbqPPo=; b=M 4oKrQLj/Di7mESLVkdJll1xB3QkYU2F5lALJ1HbElk7GPwFbX/0CB8820wqA+3IG YMGGMonmJQ/8ZLPtH/zmRae2b0m8WR9KG6I5L7whJESwmuv7ydDAylnBqdJHosux sAj8Wq96sLhq+6Nzra9RfmAZGuDYyqjN/9Ds/rrrwF8jeDbsdMA3m2t9fOKzc1zH 9px5TDxRHN/FYLoh9UCTN/kKT8kLpfvSsA1lYbvxWpglFGErGeVq90cHSxM5POg1 l8/fLPK56mqZUx30cvMdpbSUZ9O5yspQSu9Jog6Ng6aw3BxjqoBgeAdLb0x4SkI9 Blfqx7wYlq+ZiepyW53HQ== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvhedrfeeftddgheegucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepkfffgggfuffvfhfhjggtgfesthejredttdefjeenucfhrhhomheptfgvsggv tggtrgcuvehrrghnuceorhgvsggvtggtrgessghsughiohdrtghomheqnecuggftrfgrth htvghrnhepjeevgfevfeehfeeiuedtjeekhfevteeghfffleevvedvfedtledvvedtjeek feeunecuffhomhgrihhnpehgihhthhhusgdrtghomhdplhhlvhhmrdhorhhgpdgushgtrd hinhgtnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomhep rhgvsggvtggtrgessghsughiohdrtghomh X-ME-Proxy: Feedback-ID: i5b994698:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Thu, 4 May 2023 08:42:31 -0400 (EDT) Message-ID: <72dead65-cc82-97ed-516d-38163df55f62@bsdio.com> Date: Thu, 4 May 2023 06:42:30 -0600 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.0 Subject: Re: [PATCH v6 00/16] BaseTools,CryptoPkg,EmulatorPkg,MdePkg,others: Delete CLANG35,CLANG38,VS2008-2013,EBC, deprecate GCC48,GCC49,GCC5, add GCC and GCCNOLTO, update CLANGDWARF To: devel@edk2.groups.io, Liming Gao , Bob Feng , Yuwei Chen , Michael D Kinney , Michael Kubacki , Sean Brogan , Chasel Chiu , Nate DeSimone , Star Zeng , Andrew Fish , Ray Ni , Leif Lindholm , Zhiguang Liu , Jian J Wang , Xiaoyu Lu , Guomin Jiang , Gua Guo , Ard Biesheuvel , Pedro Falcato , Gerd Hoffmann , mhaeuser@posteo.de References: <20230501160839.884391-1-rebecca@bsdio.com> From: "Rebecca Cran" In-Reply-To: <20230501160839.884391-1-rebecca@bsdio.com> Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit I'd really like to get this merged: we're running out of time for the upcoming stable tag. I'm still waiting on reviews for the CryptoPkg, EmulatorPkg and IntelFsp2Pkg changes. -- Rebecca Cran On 5/1/23 10:08, 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. > > - Mark GCC48, GCC49 and GCC5 as deprecated. > > - Add GCC and GCCNOLTO toolchain definitions. > > - 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. > > Personal GitHub PR: https://github.com/tianocore/edk2/pull/4240 > GitHub branch: https://github.com/bcran/edk2/tree/clangdwarf > > Note: CI is now passing. > > 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 > update their Conf/tools_def.txt. > > Changes from v2 to v3 > ===================== > > - Keep GCC48, GCC49 and GCC5 but mark them deprecated, including with > warnings at the start and end of running a build. > > - Dropped the commit fixing the IA32 build of UefiPayloadPkg, because it > causes CI to fail due to building both IA32 and X64 at the same time: > > "Module built under multiple ARCHs [IA32, X64]. Not able to determine which output to put into flash for Module..." > > - Added more information to the VERSION section in tools_def.template. > > - Various fixes to the CLANGDWARF flags. > > Changes from v3 to v4 > ===================== > > - Use lld for -a ARM -t CLANGDWARF. > > Changes from v4 to v5 > ===================== > > - Fixed CLANGDWARF on Windows when llvm is installed with the llvm.org > installer and isn't installed in Visual Studio: use llvm-objcopy and the > tools prefix. > > - Added flags for GCCNOLTO to UnitTestFrameworkPkgHost.dsc.inc > > - Added GCC flags to CryptoPkg/Library/OpensslLib/*.inf > > Changes from v5 to v6 > ===================== > > - Reworked CLANGDWARF toolchain definitions. I made mistakes when updating > them and accidentally changed the IA32 and X64 definitions. > > - Dropped the changes to build.py. The edk2-BaseTools repo should be > updated first, and _then_ the edk2 BaseTools can be changed. > > - Split the updates to CryptoPkg build flags into separate commits, > copying the CLANGPDB flags to CLANGDWARF. > > - Fixed non-ASCII character in 5/16 commit message. > > - Removed trailing whitespace in tools_def.template. > > Rebecca Cran (16): > 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 > BaseTools: Update VS toolchain descriptions in tools_def.txt.template > Add GCC and GCCNOLTO toolchains to tools_def.txt and update packages > BaseTools/Conf/tools_def.template: Add section for deprecated > toolchains > BaseTools: Add ARM and AARCH64 CLANGDWARF support in > tools_def.template > BaseTools: As with CLANGDWARF IA32 and X64, use lld for ARM and > AARCH64 > BaseTools: Remove CLANG35 toolchain from tools_def.template > BaseTools: Delete CLANG38 from tools_def.template > BaseTools/Conf/tools_def.template: Bump VERSION to 3.00 > CryptoPkg: Delete CLANG35 and CLANG38 build flags > CryptoPkg: Copy CLANGPDB build flags to CLANGDWARF > > UnitTestFrameworkPkg/UnitTestFrameworkPkgHost.dsc.inc | 4 + > IntelFsp2Pkg/Tools/Tests/QemuFspPkg.dsc | 8 +- > 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 | 4 +- > CryptoPkg/Library/OpensslLib/OpensslLibAccel.inf | 4 +- > CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf | 4 +- > CryptoPkg/Library/OpensslLib/OpensslLibFull.inf | 4 +- > CryptoPkg/Library/OpensslLib/OpensslLibFullAccel.inf | 5 +- > EmulatorPkg/Unix/Host/Host.inf | 2 + > MdePkg/Include/Ia32/ProcessorBind.h | 8 +- > MdePkg/Include/X64/ProcessorBind.h | 8 +- > BaseTools/Conf/tools_def.template | 1909 +++++--------------- > 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 +- > 23 files changed, 530 insertions(+), 1621 deletions(-) >