Jordan: > -----Original Message----- > From: Justen, Jordan L > Sent: Monday, May 20, 2019 4:15 AM > To: Ard Biesheuvel <ard.biesheu...@linaro.org>; Gao, Liming > <liming....@intel.com>; edk2-devel-groups-io <devel@edk2.groups.io> > Subject: Re: [edk2-devel] [Patch 0/7] Add new CLANG8ELF tool chain for new > LLVM/CLANG8 > > On 2019-04-27 17:55:02, Liming Gao wrote: > > >-----Original Message----- > > >From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > > >Sent: Saturday, April 27, 2019 12:33 AM > > > > > > > > >This series confuses me. The existing CLANGxx toolchains already use > > >GenFw and ELF to PE/COFF conversion, so the name CLANG8ELF is > > >misleading. > > > > > LLVM/CLANG8.0 compiler supports to generate PE image or ELF image. > > This tool chain is to generate ELF image and be converted to PE > > image. I am investigating another tool chain with CLANG8.0 to > > directly generate PE image. To differentiate them, I use the tool > > chain name CLANG8ELF and CLANG8PE for them. > > Assuming CLANG8ELF and CLANG8PE were functional, would both be needed? > It kind of sounds like this a half-finished investigation. > One point is that they will generate the different debug format symbols (WinDBG or GDB). I have not done the full investigation to generate the different debug format symbols in single tool chain.
> I'm guessing that if CLANG8PE produces equal or better code, then it > would be preferred. > > Therefore, shouldn't we just finish the investigation, and add a > single CLANG8 toolchain at the end? Or, maybe to reflect that it > mostly uses the LLVM tool stack we could name it LLVM8. I also prefer single CLANG8 tool chain. I will collect more information and see whether it is possible. Now, I would like to add this tool chain for the developer evaluation. Because I can't address all comments now, I will remove this feature from Q2 stable tag. I will add it into edk2-staging first. > > -Jordan -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#41053): https://edk2.groups.io/g/devel/message/41053 Mute This Topic: https://groups.io/mt/31354044/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
