Re: [edk2] [PATCH v5 0/8] BaseTools: add support for GCC5 in LTO mode

2016-08-02 Thread Michael Zimmermann
sry but I have to retest everything since I aciidentally checked out a wrong revision. will report back later. On Tue, Aug 2, 2016 at 4:39 PM, Michael Zimmermann wrote: > Sry for the late reply but I tried your latest gcc-lto branch to make sure > my answers are still

Re: [edk2] [PATCH v5 0/8] BaseTools: add support for GCC5 in LTO mode

2016-08-02 Thread Ard Biesheuvel
On 2 August 2016 at 16:39, Michael Zimmermann wrote: > Sry for the late reply but I tried your latest gcc-lto branch to make sure > my answers are still correct. > I tested it with all configurations(StdLib:X64/ARM, Ovmf:X64, ArmVirtPkg:ARM > with both RELEASE and

Re: [edk2] [PATCH v5 0/8] BaseTools: add support for GCC5 in LTO mode

2016-08-02 Thread Michael Zimmermann
Sry for the late reply but I tried your latest gcc-lto branch to make sure my answers are still correct. I tested it with all configurations(StdLib:X64/ARM, Ovmf:X64, ArmVirtPkg:ARM with both RELEASE and DEBUG). In StdLib are many warnings actually. That's because of -Wunused-const-variable and

Re: [edk2] [PATCH v5 0/8] BaseTools: add support for GCC5 in LTO mode

2016-08-02 Thread Ard Biesheuvel
On 2 August 2016 at 15:55, Michael Zimmermann wrote: > btw, gcc6 seems to work fine too with these patches(it did with the GCC49 > configs too on both ARM and x86/x64). > Thanks for the data points. Do you see any warnings during the build?

Re: [edk2] [PATCH v5 0/8] BaseTools: add support for GCC5 in LTO mode

2016-08-02 Thread Michael Zimmermann
btw, gcc6 seems to work fine too with these patches(it did with the GCC49 configs too on both ARM and x86/x64). Thanks Michael On Tue, Aug 2, 2016 at 1:42 PM, Ard Biesheuvel wrote: > On 2 August 2016 at 13:41, Shi, Steven wrote: > > Ard, > >

Re: [edk2] [PATCH v5 0/8] BaseTools: add support for GCC5 in LTO mode

2016-08-02 Thread Ard Biesheuvel
On 2 August 2016 at 13:41, Shi, Steven wrote: > Ard, > Thank you to check in GCC5! > My pleasure. Thanks to you for the comments, discussion and review feedback. -- Ard. ___ edk2-devel mailing list edk2-devel@lists.01.org

Re: [edk2] [PATCH v5 0/8] BaseTools: add support for GCC5 in LTO mode

2016-08-02 Thread Shi, Steven
Ard, Thank you to check in GCC5! Steven Shi Intel\SSG\STO\UEFI Firmware Tel: +86 021-61166522 iNet: 821-6522 > -Original Message- > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Tuesday, August 02, 2016 5:03 PM > To: Shi, Steven ; Zhu, Yonghong

Re: [edk2] [PATCH v5 0/8] BaseTools: add support for GCC5 in LTO mode

2016-08-02 Thread Ard Biesheuvel
On 2 August 2016 at 12:57, Laszlo Ersek wrote: > On 08/02/16 11:03, Ard Biesheuvel wrote: >> On 1 August 2016 at 10:01, Ard Biesheuvel wrote: >>> This v5 to introduce GCC5 is now a 8 piece series, including some >>> preparatory cleanup patches that

Re: [edk2] [PATCH v5 0/8] BaseTools: add support for GCC5 in LTO mode

2016-08-02 Thread Ard Biesheuvel
On 1 August 2016 at 10:01, Ard Biesheuvel wrote: > This v5 to introduce GCC5 is now a 8 piece series, including some > preparatory cleanup patches that allow all GCC4x and CLANG35 toolchains > to switch to using 'gcc' as the linker. This allows us to get rid of > the

Re: [edk2] [PATCH v5 0/8] BaseTools: add support for GCC5 in LTO mode

2016-08-01 Thread Shi, Steven
Ard, Where can I check out your v5 patch? Steven Shi Intel\SSG\STO\UEFI Firmware Tel: +86 021-61166522 iNet: 821-6522 > -Original Message- > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Monday, August 01, 2016 4:01 PM > To: Shi, Steven ; Zhu,

Re: [edk2] [PATCH v5 0/8] BaseTools: add support for GCC5 in LTO mode

2016-08-01 Thread Ard Biesheuvel
On 1 August 2016 at 16:01, Shi, Steven wrote: > Ard, > Where can I check out your v5 patch? > https://git.linaro.org/people/ard.biesheuvel/uefi-next.git/shortlog/refs/heads/gcc5-lto-v5 or git://git.linaro.org/people/ard.biesheuvel/uefi-next.git gcc5-lto-v5 Thanks, Ard.

[edk2] [PATCH v5 0/8] BaseTools: add support for GCC5 in LTO mode

2016-08-01 Thread Ard Biesheuvel
This v5 to introduce GCC5 is now a 8 piece series, including some preparatory cleanup patches that allow all GCC4x and CLANG35 toolchains to switch to using 'gcc' as the linker. This allows us to get rid of the wrapper script to marshall ld arguments in order to make them understandable by gcc,