On Wed, 10 May 2023 at 14:46, Rebecca Cran <rebe...@bsdio.com> wrote: > > On 5/10/23 06:37, Ard Biesheuvel wrote: > > On Wed, 10 May 2023 at 14:31, Rebecca Cran <rebe...@bsdio.com> wrote: > >> This reverts commit 11f62f4cc09f16d265da1a737dabfd8ed65f8c00. > >> > >> While GCC uses objcopy for the OBJCOPY command, it's not needed for the > >> CLANGDWARF toolchain and can be left as echo. > >> > > Are you sure you want to change this for IA32 and X86 as well? I'm not > > sure what the significance is, I suppose it sets the debuglink on the > > copies of the dll files? > > Looking back to the tools_def.template from edk2-stable202302, we set > OBJCOPY_PATH for CLANG38 IA32 and X64, but not for CLANGPDB or CLANGDWARF. > > I was thinking it's safer to keep existing behavior for now. >
Existing behavior of CLANGDWARF for X86 and IA32, right? Yeah that makes sense. Reviewed-by: Ard BIesheuvel <a...@kernel.org> -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#104558): https://edk2.groups.io/g/devel/message/104558 Mute This Topic: https://groups.io/mt/98804529/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
