Re: [edk2-devel] [PATCH 3/9] BaseTools: Update CLANGDWARF toolchain and remove CLANG35 and CLANG38

2023-03-27 Thread Rebecca Cran
m>>; Leif Lindholm > mailto:quic_llind...@quicinc.com>>; Ard Biesheuvel > mailto:ardb+tianoc...@kernel.org>>; Justen, Jordan L > mailto:jordan.l.jus...@intel.com>>; Gerd Hoffmann mailto:kra...@redhat.com>> > Subject: Re: [edk2-devel] [PA

Re: [edk2-devel] [PATCH 3/9] BaseTools: Update CLANGDWARF toolchain and remove CLANG35 and CLANG38

2023-03-23 Thread Ard Biesheuvel
On Thu, 23 Mar 2023 at 10:04, Ard Biesheuvel wrote: > > On Thu, 23 Mar 2023 at 02:30, Rebecca Cran wrote: > > > > 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

Re: [edk2-devel] [PATCH 3/9] BaseTools: Update CLANGDWARF toolchain and remove CLANG35 and CLANG38

2023-03-23 Thread Ard Biesheuvel
On Thu, 23 Mar 2023 at 02:30, Rebecca Cran wrote: > > 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

Re: [edk2-devel] [PATCH 3/9] BaseTools: Update CLANGDWARF toolchain and remove CLANG35 and CLANG38

2023-03-22 Thread Rebecca Cran
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. Just to

Re: [edk2-devel] [PATCH 3/9] BaseTools: Update CLANGDWARF toolchain and remove CLANG35 and CLANG38

2023-03-22 Thread Rebecca Cran
: Guo, Gua ; devel@edk2.groups.io; Kinney, Michael D ; Gao, Liming ; Liu, Zhiguang ; Feng, Bob C ; Chen, Christine ; Andrew Fish ; Leif Lindholm ; Ard Biesheuvel ; Justen, Jordan L ; Gerd Hoffmann Subject: Re: [edk2-devel] [PATCH 3/9] BaseTools: Update CLANGDWARF toolchain and remove CLANG35 a

Re: [edk2-devel] [PATCH 3/9] BaseTools: Update CLANGDWARF toolchain and remove CLANG35 and CLANG38

2023-03-22 Thread Rebecca Cran
I'm not seeing this failure. Could you tell me what the following commands print, please: where clang where lld Mine are: C:\Users\bcran\Documents\src\uefi\edk2>where clang C:\Program Files\LLVM\bin\clang.exe C:\Program Files (x86)\Microsoft Visual

Re: [edk2-devel] [PATCH 3/9] BaseTools: Update CLANGDWARF toolchain and remove CLANG35 and CLANG38

2023-03-22 Thread Ard Biesheuvel
On Wed, 22 Mar 2023 at 14:03, Gerd Hoffmann wrote: > > On Wed, Mar 22, 2023 at 01:32:01PM +0100, Ard Biesheuvel wrote: > > On Wed, 22 Mar 2023 at 13:28, Rebecca Cran wrote: > > > > > > On 3/22/23 5:49 AM, Ard Biesheuvel wrote: > > > > > > > The reason I added CLANG3x support for ARM in the past

Re: [edk2-devel] [PATCH 3/9] BaseTools: Update CLANGDWARF toolchain and remove CLANG35 and CLANG38

2023-03-22 Thread Gerd Hoffmann
On Wed, Mar 22, 2023 at 01:32:01PM +0100, Ard Biesheuvel wrote: > On Wed, 22 Mar 2023 at 13:28, Rebecca Cran wrote: > > > > 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

Re: [edk2-devel] [PATCH 3/9] BaseTools: Update CLANGDWARF toolchain and remove CLANG35 and CLANG38

2023-03-22 Thread Ard Biesheuvel
On Wed, 22 Mar 2023 at 13:28, Rebecca Cran wrote: > > 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

Re: [edk2-devel] [PATCH 3/9] BaseTools: Update CLANGDWARF toolchain and remove CLANG35 and CLANG38

2023-03-22 Thread Rebecca Cran
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

Re: [edk2-devel] [PATCH 3/9] BaseTools: Update CLANGDWARF toolchain and remove CLANG35 and CLANG38

2023-03-22 Thread Ard Biesheuvel
On Wed, 22 Mar 2023 at 02:30, Rebecca Cran wrote: > > Update the CLANGDWARF toolchain definition with the settings from > CLANG38, and delete the CLANG35 and CLANG38 toolchains. > > The existing CLANGDWARF toolchain definition used ld.lld, but this > causes the following linker errors when