From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by ml01.01.org (Postfix) with ESMTP id 02F031A1E2E for ; Mon, 1 Aug 2016 07:01:33 -0700 (PDT) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga101.jf.intel.com with ESMTP; 01 Aug 2016 07:01:24 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.28,455,1464678000"; d="scan'208";a="1006311569" Received: from fmsmsx107.amr.corp.intel.com ([10.18.124.205]) by orsmga001.jf.intel.com with ESMTP; 01 Aug 2016 07:01:22 -0700 Received: from fmsmsx158.amr.corp.intel.com (10.18.116.75) by fmsmsx107.amr.corp.intel.com (10.18.124.205) with Microsoft SMTP Server (TLS) id 14.3.248.2; Mon, 1 Aug 2016 07:01:22 -0700 Received: from shsmsx101.ccr.corp.intel.com (10.239.4.153) by fmsmsx158.amr.corp.intel.com (10.18.116.75) with Microsoft SMTP Server (TLS) id 14.3.248.2; Mon, 1 Aug 2016 07:01:21 -0700 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.147]) by SHSMSX101.ccr.corp.intel.com ([169.254.1.8]) with mapi id 14.03.0248.002; Mon, 1 Aug 2016 22:01:20 +0800 From: "Shi, Steven" To: Ard Biesheuvel , "Zhu, Yonghong" , "Gao, Liming" , "Justen, Jordan L" , "edk2-devel@lists.01.org" CC: "leif.lindholm@linaro.org" , "lersek@redhat.com" Thread-Topic: [PATCH v5 0/8] BaseTools: add support for GCC5 in LTO mode Thread-Index: AQHR68sHy1Dhvz07h06dwb5hyY9qgqA0Im5Q Date: Mon, 1 Aug 2016 14:01:19 +0000 Message-ID: <06C8AB66E78EE34A949939824ABE2B3103382886@shsmsx102.ccr.corp.intel.com> References: <1470038497-2536-1-git-send-email-ard.biesheuvel@linaro.org> In-Reply-To: <1470038497-2536-1-git-send-email-ard.biesheuvel@linaro.org> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiZGZjMGEzMTgtZWJjZS00ZTYwLWJhMzMtNjg4NjgxNTgyYTAzIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX0lDIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE1LjkuNi42IiwiVHJ1c3RlZExhYmVsSGFzaCI6IjFONUt3aTh2cFBjV2J1dFN5SHd4MDlCTTRhdXFkaHhweVNaUjErdFI0cGM9In0= x-ctpclassification: CTP_IC x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [PATCH v5 0/8] BaseTools: add support for GCC5 in LTO mode X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 01 Aug 2016 14:01:34 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable 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.biesheuvel@linaro.org] > Sent: Monday, August 01, 2016 4:01 PM > To: Shi, Steven ; Zhu, Yonghong > ; Gao, Liming ; Justen, > Jordan L ; edk2-devel@lists.01.org > Cc: leif.lindholm@linaro.org; lersek@redhat.com; Ard Biesheuvel > > Subject: [PATCH v5 0/8] BaseTools: add support for GCC5 in LTO mode >=20 > 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, which is fragile and likely to cause problems in > the future. >=20 > Since there appears to be a natural split between the 'legacy' GCC > toolchains UNIXGCC, ELFGCC, and CYGGCC[xASL], both in term of supported > architectures [IA32, X64, IPF] vs [IA32, X64, ARM, AARCH64], and in > terms of maintenance, these toolchains are not moved to using 'gcc' as > the linker, and instead, a new BUILDRULEFAMILY is introduced called GCCLD > that will retain the old behavior. >=20 > The result is that GCC5 can align much more closely with its predecessors= , > making the expected maintenance burden of supporting GCC back to v4.4 > much lower. >=20 > Changes since v4: > - added patch to use 'protected' visibility only for the libraries that > define the module entry points (_ModuleEntryPoint), to prevent them fro= m > being optimized away by the LTO routines > - added Jordan's ack/RBs > - add some extra comments to tools_def.template (#8) >=20 > Changes since v3: > - like Steven does in his GCC5LTO patch, add -fno-builtin to IA32 and X64 > CC_FLAGS; this addresses a build issue reported by Liming > - add -Os the the linker flags as well, for AARCH64 this does not seem to > make > a difference, but it is arguably correct since the LTO processing at li= nk > time involves code generation as well > - add Laszlo's ack to #2 > - new patch #6 to omit the autogenerated build-id from the PE/COFF binary >=20 > Changes since v2: > - add license headers to LTO glue files for ARM and AARCH64 (#5) > - get rid of lto-ld-wrapper script >=20 > Ard Biesheuvel (8): > BaseTools CLANG35: drop problematic use-movt and save-temps options > ArmVirtPkg/ArmVirtPrePiUniCoreRelocatable: ignore .hash and .note > sections > BaseTools UNIXGCC ELFGCC CYGGCC: clone GCC build rule family into > GCCLD > BaseTools GCC: use 'gcc' as the linker command for GCC44 and later > ArmPkg: add prebuilt glue binaries for GCC5 LTO support > BaseTools GCC: drop GNU notes section from EFI image > MdePkg GCC/X64: avoid 'hidden' visibility for module entry points > BaseTools GCC: introduce GCC5 toolchain to support GCC v5.x in LTO > mode >=20 > ArmPkg/GccLto/liblto-aarch64.a |= Bin 0 -> 1016 bytes > ArmPkg/GccLto/liblto-aarch64.s |= 27 ++ > ArmPkg/GccLto/liblto-arm.a |= Bin 0 -> 2096 bytes > ArmPkg/GccLto/liblto-arm.s |= 61 ++++ > ArmVirtPkg/PrePi/ArmVirtPrePiUniCoreRelocatable.inf |= 2 +- > ArmVirtPkg/PrePi/Scripts/PrePi-PIE.lds |= 3 + > BaseTools/Conf/build_rule.template |= 31 +- > BaseTools/Conf/tools_def.template |= 350 > +++++++++++++++----- > BaseTools/Scripts/GccBase.lds |= 6 + > EmulatorPkg/Unix/Host/Host.inf |= 6 +- > MdePkg/Include/X64/ProcessorBind.h |= 9 +- > MdePkg/Library/DxeCoreEntryPoint/DxeCoreEntryPoint.inf |= 2 + > MdePkg/Library/PeiCoreEntryPoint/PeiCoreEntryPoint.inf |= 2 + > MdePkg/Library/PeimEntryPoint/PeimEntryPoint.inf |= 2 + > MdePkg/Library/UefiApplicationEntryPoint/UefiApplicationEntryPoint.inf | > 2 + > MdePkg/Library/UefiDriverEntryPoint/UefiDriverEntryPoint.inf |= 2 + > 16 files changed, 396 insertions(+), 109 deletions(-) > create mode 100644 ArmPkg/GccLto/liblto-aarch64.a > create mode 100644 ArmPkg/GccLto/liblto-aarch64.s > create mode 100644 ArmPkg/GccLto/liblto-arm.a > create mode 100644 ArmPkg/GccLto/liblto-arm.s >=20 > -- > 2.7.4