From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) by mx.groups.io with SMTP id smtpd.web10.41359.1679488091938455478 for ; Wed, 22 Mar 2023 05:28:12 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="signature has expired" header.i=@bsdio.com header.s=fm3 header.b=WNZIxFSe; spf=pass (domain: bsdio.com, ip: 66.111.4.27, mailfrom: rebecca@bsdio.com) Received: from compute3.internal (compute3.nyi.internal [10.202.2.43]) by mailout.nyi.internal (Postfix) with ESMTP id D5FB15C00D7; Wed, 22 Mar 2023 08:28:10 -0400 (EDT) Received: from mailfrontend2 ([10.202.2.163]) by compute3.internal (MEProxy); Wed, 22 Mar 2023 08:28:10 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdio.com; h=cc :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=fm3; t= 1679488090; x=1679574490; bh=MIl6dPuSDv64wwrsExYFQg0AfJ7L1OSjsyS WsSGplOM=; b=WNZIxFSeI0luYgTSuAv4Lhw8Nx/Qnnw1SPoP5fK70zAMoMW1yWc lXrjn3goF6yf3PnZkcWt+PulwEbFezTC3AWxf1iO5oqSAJPh7tV2JeKARcq/FB/I DU58rOHRfx5MvGthH3G5RlehftTrJcrJ0LFAecG093V2gUFjPbdm6Y1xU11QVkIS q3cF9fVlEZCSx0LAepZARyxa6cdFJp4j1suFNImYLub9TnEX1zHj8Nwi0dkQDGjb gOylLg36UvcsyLaenqbHdP0G/K1tT3tuVuRxoR/gB9g6XEoCfNwq8qUeOUYRy+Ir tsuq+oR1ptwQn/n5BjHztcYbMLIBrr+l1oQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc: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=fm2; t= 1679488090; x=1679574490; bh=MIl6dPuSDv64wwrsExYFQg0AfJ7L1OSjsyS WsSGplOM=; b=eWJetoGE6M+0RwosunFnU6g6IyUlSo9zyYWokdNzCllYhnmcUK4 s/1WL1XkoAhx5bI1B9w+9+lCs27gptovJmrNZ9CF7gcVYLqjfpsK/Iftr286wFiX n+VkuU960JTZOKfdY5jW82H1xxKG2lBU+GkAsCS8ACizdvC7LjoEoWsZcdLGn3Pv w1Xl2GVuwYrlO0A6Giz2B7HYDbA2k7Jr62NwBhlhkeJwEG9KVR2GeDRRYH4/0A4L M3XenpRClg0M57jZJsnKAAerdzEMxZT/a4T6W9GmLdIGhC/jTuoK3HI5PTRLhPRe bNBJFKH7d5vKiy4lxIHVNE+6S8vW4Bznq+A== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvhedrvdegvddggedvucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepkfffgggfuffvvehfhfgjtgfgsehtjeertddtfeejnecuhfhrohhmpeftvggs vggttggrucevrhgrnhcuoehrvggsvggttggrsegsshguihhordgtohhmqeenucggtffrrg htthgvrhhnpeehleekgfdutedugeehgeffgffhteefhffhteetfeevhefgvdejhedthfff ieffleenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpe hrvggsvggttggrsegsshguihhordgtohhm X-ME-Proxy: Feedback-ID: i5b994698:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Wed, 22 Mar 2023 08:28:08 -0400 (EDT) Message-ID: Date: Wed, 22 Mar 2023 06:28:07 -0600 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.9.0 Subject: Re: [edk2-devel] [PATCH 3/9] BaseTools: Update CLANGDWARF toolchain and remove CLANG35 and CLANG38 To: devel@edk2.groups.io, ardb@kernel.org Cc: 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> <20230322013040.3322865-4-rebecca@bsdio.com> From: "Rebecca Cran" In-Reply-To: Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 3/22/23 5:49 AM, Ard Biesheuvel wrote: > The reason I added CLANG3x support for ARM in the past is to ensure > compatibility with the ARM proprietary, Clang based toolchain. At the > time, we went with GNU ld, but I would actually prefer if we could > make this work with LLD as well. > > I can work around this issue locally by doing > > --- a/OvmfPkg/OvmfPkgX64.dsc > +++ b/OvmfPkg/OvmfPkgX64.dsc > @@ -297,7 +297,7 @@ > PeiServicesLib|MdePkg/Library/PeiServicesLib/PeiServicesLib.inf > PeiServicesTablePointerLib|MdePkg/Library/PeiServicesTablePointerLibIdt/PeiServicesTablePointerLibIdt.inf > MemoryAllocationLib|MdePkg/Library/PeiMemoryAllocationLib/PeiMemoryAllocationLib.inf > -!if $(TOOL_CHAIN_TAG) == "XCODE5" > +!if $(TOOL_CHAIN_TAG) == "XCODE5" || $(TOOL_CHAIN_TAG) == "CLANGDWARF" > CpuExceptionHandlerLib|UefiCpuPkg/Library/CpuExceptionHandlerLib/Xcode5SecPeiCpuExceptionHandlerLib.inf > !else > > Can you please check whether this works for you as well? Thanks, that works here too! Do we still need to keep CLANG35 and CLANG38 toolchains for compatibility with the ARM toolchain? Or have things moved on so they _can_ be removed? -- Rebecca Cran