From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=pass header.i=@linaro.org header.s=google header.b=S9l/5B5A; spf=pass (domain: linaro.org, ip: 209.85.166.174, mailfrom: ard.biesheuvel@linaro.org) Received: from mail-it1-f174.google.com (mail-it1-f174.google.com [209.85.166.174]) by groups.io with SMTP; Fri, 26 Apr 2019 09:33:35 -0700 Received: by mail-it1-f174.google.com with SMTP id s3so6848312itk.1 for ; Fri, 26 Apr 2019 09:33:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to; bh=foKfeHL9CyeRMqCLpoXvDWtimRbmTd/vOpCcIDQdU6I=; b=S9l/5B5AL+Gma8AhDaXHk0zGdq2pxfVxvdvuhC4065ZMXsVvvtB3onrHnEXgOeE7Yu XB2aIq2Da2yBnQnjMI6aa3FQ5mQFaY/nfMkbuRdKAt1751ERNV+/8kEjiErqz5h9M/yr jhdn5Ya2PjQPcz6ldQtxowOrxpS/i3bZTlb7xNWC+10dFElJj+p+c3qC2BBIATx5RUTO RHBmw/414moNFEo+W12xxyYGV9ZGvJbvPc2T0Q4+yMK5tPSXZMGVhEON5eUpclpVc8Fy 0Yyo4l5Tb26006ALJs86uTN4chCazIrZzlTCmASwxsfPlbByEdqzWuqrGy/IEyGA6VjW f0Fg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to; bh=foKfeHL9CyeRMqCLpoXvDWtimRbmTd/vOpCcIDQdU6I=; b=RLqHtIdB/XqwsMaXZFho+5GYxvcBO6xhimFkf1etPeUOiq/s8ShpJk+m/5R6GTtBaL zdeFA943VeM1vj3doMoJmbxq+Dn5Ysr5sc1+5B3LhXlNubPjl5BYAThkYNobjwKiDe6b dBwwmQ/73MjJStcpT5vqvNDmZnl9K8c7Mh/VwKhZFQF6P4UUzj529Hh2jr6X0XpEt57f oOcpguOfA8YcqAVmsHvCyBe/fCfBmlVafuccDFs52S/2c5ddwHbQHUZ3c2QDn6l6tX2O LVrnGWwFXKuuRhKygqsd53+928SJ+e/7fC0WvHq3zNGv0/3syv0WaRelCU5lCkM3Y9JI sOKg== X-Gm-Message-State: APjAAAVAXT1E+qMTx5aIvf2K3qnQbD2QpPn1My2j9kYg6uo24wUnzSUs /F1Fq/91JCo1pngIGwZbzBAYA8jkLYjA9Q3dtK33pqOr2/g= X-Google-Smtp-Source: APXvYqzlpRIdP5a91Ju/CqTZoX8N/0yf3V6z/WMq5p/6Ex9L78Z9gd0W4PAwQtvkq7Z3jI6muM8zetxMg5BybMwPaf0= X-Received: by 2002:a24:59c1:: with SMTP id p184mr9982164itb.158.1556296414719; Fri, 26 Apr 2019 09:33:34 -0700 (PDT) MIME-Version: 1.0 References: <20190426144242.19024-1-liming.gao@intel.com> In-Reply-To: <20190426144242.19024-1-liming.gao@intel.com> From: "Ard Biesheuvel" Date: Fri, 26 Apr 2019 18:33:21 +0200 Message-ID: Subject: Re: [edk2-devel] [Patch 0/7] Add new CLANG8ELF tool chain for new LLVM/CLANG8 To: edk2-devel-groups-io , "Gao, Liming" Content-Type: text/plain; charset="UTF-8" On Fri, 26 Apr 2019 at 16:43, Liming Gao wrote: > > BZ:https://bugzilla.tianocore.org/show_bug.cgi?id=1603 > LLVM/CLANG8 formal release http://releases.llvm.org/download.html#8.0.0 > It can be downloaded and installed in Windows/Linux/Mac OS. > > CLANG8ELF tool chain is added to generate ELF image, and convert to PE/COFF. > On Windows OS, set CLANG_HOST_BIN=n, set CLANG8_BIN=LLVM installed directory > CLANG_HOST_BIN is used CLANG_HOST_PREFIX. Prefix n is for nmake. > For example: > set CLANG_HOST_BIN=n > set CLANG8_BIN=C:\Program Files\LLVM\bin\ > On Linux/Mac, export CLANG8_BIN=LLVM installed directory, CLANG_HOST_BIN is > not required, because there is no prefix for make. > For example: > export CLANG8_BIN=/home/clang8/bin/ > > This tool chain can be used to compile the firmware code. On windows OS, > Visual Studio is still required to compile BaseTools C tools and > provide nmake.exe for makefile. On Linux/Mac OS, gcc is used to compile > BaseTools C tools. make is used for makefile. > > This tool chain is verified on OVMF Ia32, X64 and Ia32X64 to boot Shell. > This tool chain is verified in Windows/Linux and Mac OS. > Hello Liming, This series confuses me. The existing CLANGxx toolchains already use GenFw and ELF to PE/COFF conversion, so the name CLANG8ELF is misleading. Also, it seems that the primary difference is using LLD instead of GNU ld, but this has nothing to do with the Clang version. What is the benefit of using LLD over GNU ld? It seems we are working around various incompatibilities, and I think this is only justified if LLD has some benefit over GNU ld. Thanks, Ard. > Liming Gao (7): > BaseTools: Add ClangBase.lds for CLANG8 tool chain with max-page-size > BaseTools GenFw: Support CLANG8ELF with conversion ELF to PE/COFF > image > BaseTools: Update build_rule.template for ASLC rule with full C flags > BaseTools: Update build_rule to skip CLANG resource section generation > BaseTools: Update tools_def.template to directly refer to AutoGen.h > BaseTools: Add new CLANG8ELF tool chain for new LLVM/CLANG8 > OvmfPkg: Update DSC/FDF to support CLANG8ELF tool chain > > BaseTools/Source/C/GenFw/Elf32Convert.c | 12 +--- > BaseTools/Source/C/GenFw/Elf64Convert.c | 11 +-- > BaseTools/Conf/build_rule.template | 8 +-- > BaseTools/Conf/tools_def.template | 121 +++++++++++++++++++++++++++++--- > BaseTools/Scripts/ClangBase.lds | 79 +++++++++++++++++++++ > OvmfPkg/OvmfPkgIa32.dsc | 4 +- > OvmfPkg/OvmfPkgIa32.fdf | 2 +- > OvmfPkg/OvmfPkgIa32X64.dsc | 4 +- > OvmfPkg/OvmfPkgIa32X64.fdf | 2 +- > OvmfPkg/OvmfPkgX64.dsc | 4 +- > OvmfPkg/OvmfPkgX64.fdf | 2 +- > 11 files changed, 213 insertions(+), 36 deletions(-) > create mode 100644 BaseTools/Scripts/ClangBase.lds > > -- > 2.13.0.windows.1 > > > >