Ray:

If the tool chain generates ELF image, its ELF image can be converted to EFI image by Edk2 GenFw tool. Then, this tool chain can be used for EFI development.

So, I propose to make CLANGDWARF tool chain generate ELF image and EFI image both. It can support the usage of ELF image generation.

Thanks

Liming

发件人: Ni, Ray <ray.ni@intel.com>
发送时间: 2021年5月28日 22:31
收件人: devel@edk2.groups.io; gaoliming@byosoft.com.cn; Shi, Steven <steven.shi@intel.com>
主题: Re: [edk2-devel] [PATCH 1/4] BaseTools: Create CLANGDWARF toolchain

Liming,

that is not the intention of this new toolchain.

we want to have a toolchain that can generate elf. or can you suggest a new name for this toolchain?

thanks,

ray

From: devel@edk2.groups.io <devel@edk2.groups.io> on behalf of gaoliming <gaoliming@byosoft.com.cn>
Sent: Friday, May 28, 2021 10:52:06 AM
To: devel@edk2.groups.io <devel@edk2.groups.io>; Shi, Steven <steven.shi@intel.com>; Ni, Ray <ray.ni@intel.com>
Subject: 回复: [edk2-devel] [PATCH 1/4] BaseTools: Create CLANGDWARF toolchain

Ray:

I would like to suggest CLANGDWARF also generate EFI image. If so, the people can use this tool chain for EFI development with DWARF format debug symbol.

In dll generation phase, CLANGDWARF still generates dll image, then copy dll image to elf image. In EFI generation phase, dll image will be converted to EFI image.

Thanks

Liming

发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Steven Shi
发送时间: 2021年5月26日 19:35
收件人: devel@edk2.groups.io; Ni, Ray <ray.ni@intel.com>
主题: Re: [edk2-devel] [PATCH 1/4] BaseTools: Create CLANGDWARF toolchain

Some suggestions:

There are many blanks appending the line “+ "$(DLINK)" -o ${dst} $(DLINK_FLAGS) @$(STATIC_LIBRARY_FILES_LIST) “

We need a toolchain dependency descirption in the BaseTools\Conf\tools_def.template. Below is the example for CLANGPDB. Please add one for the CLANGDWARF.

# CLANGPDB -Linux, Windows, Mac- Requires:

# Clang 9 or above from http://releases.llvm.org/

# Optional:

# Required to compile nasm source:

# nasm compiler from

# NASM -- http://www.nasm.us/

# Required to build platforms or ACPI tables:

# Intel(r) ACPI Compiler from

# https://acpica.org/downloads

We could merge the CLANG38 and the CLANGDWARF toolchains together in the future.

Thanks

Steven Shi

> -----Original Message-----

> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Ni, Ray

> Sent: Wednesday, May 26, 2021 4:23 PM

> To: devel@edk2.groups.io

> Subject: [edk2-devel] [PATCH 1/4] BaseTools: Create CLANGDWARF

> toolchain

> Signed-off-by: Ray Ni <ray.ni@intel.com>

> ---

> BaseTools/Conf/build_rule.template | 10 ++-

> BaseTools/Conf/tools_def.template | 98

> ++++++++++++++++++++++++++++++

> 2 files changed, 107 insertions(+), 1 deletion(-)

> diff --git a/BaseTools/Conf/build_rule.template

> b/BaseTools/Conf/build_rule.template

> index 1395792cd6..e69f963cc8 100755

> --- a/BaseTools/Conf/build_rule.template

> +++ b/BaseTools/Conf/build_rule.template

> @@ -293,7 +293,10 @@

> <ExtraDependency>

> $(MAKE_FILE)

> - <OutputFile>

> + <OutputFile.CLANGDWARF>

> + $(DEBUG_DIR)(+)$(MODULE_NAME).elf

> +

> + <OutputFile.MSFT, OutputFile.INTEL, OutputFile.GCC>

> $(DEBUG_DIR)(+)$(MODULE_NAME).dll

> <Command.MSFT, Command.INTEL>

> @@ -303,6 +306,11 @@

> <Command.CLANGPDB>

> "$(DLINK)" /OUT:${dst} $(DLINK_FLAGS) $(DLINK_SPATH)

> @$(STATIC_LIBRARY_FILES_LIST) $(DLINK2_FLAGS)

> + <Command.CLANGDWARF>

> + "$(DLINK)" -o ${dst} $(DLINK_FLAGS) @$(STATIC_LIBRARY_FILES_LIST)

> + "$(CP)" ${dst} $(DEBUG_DIR)(+)$(MODULE_NAME).debug

> + "$(OBJCOPY)" --strip-unneeded -R .eh_frame ${dst}

> +

> <Command.GCC>

> "$(DLINK)" -o ${dst} $(DLINK_FLAGS) -Wl,--start-

> group,@$(STATIC_LIBRARY_FILES_LIST),--end-group $(CC_FLAGS)

> $(DLINK2_FLAGS)

> "$(OBJCOPY)" $(OBJCOPY_FLAGS) ${dst}

> diff --git a/BaseTools/Conf/tools_def.template

> b/BaseTools/Conf/tools_def.template

> index 498696e583..390e297cc1 100755

> --- a/BaseTools/Conf/tools_def.template

> +++ b/BaseTools/Conf/tools_def.template

> @@ -2828,6 +2828,104 @@ NOOPT_CLANGPDB_X64_DLINK_FLAGS =

> /NOLOGO /NODEFAULTLIB /IGNORE:4001 /OPT:RE

> NOOPT_CLANGPDB_X64_DLINK2_FLAGS =

> NOOPT_CLANGPDB_X64_GENFW_FLAGS = --keepexceptiontable

> +#########################################################

> ###########################

> +#

> +# CLANGDWARF - This configuration is used to compile under

> Windows/Linux/Mac to produce

> +# ELF binaries using LLVM/Clang/LLD with Link Time Optimization enabled

> +#

> +#########################################################

> ###########################

> +*_CLANGDWARF_*_*_FAMILY = GCC

> +*_CLANGDWARF_*_*_BUILDRULEFAMILY = CLANGDWARF

> +*_CLANGDWARF_*_MAKE_PATH = ENV(CLANG_HOST_BIN)make

> +*_CLANGDWARF_*_*_DLL = ENV(CLANGPDB_DLL)

> +*_CLANGDWARF_*_ASL_PATH = DEF(UNIX_IASL_BIN)

> +

> +*_CLANGDWARF_*_APP_FLAGS =

> +*_CLANGDWARF_*_ASL_FLAGS = DEF(DEFAULT_WIN_ASL_FLAGS)

> +*_CLANGDWARF_*_ASL_OUTFLAGS =

> DEF(DEFAULT_WIN_ASL_OUTFLAGS)

> +*_CLANGDWARF_*_ASLDLINK_FLAGS = DEF(MSFT_ASLDLINK_FLAGS)

> +*_CLANGDWARF_*_DEPS_FLAGS = DEF(GCC_DEPS_FLAGS)

> +

> +DEFINE CLANGDWARF_IA32_PREFIX = ENV(CLANG_BIN)

> +DEFINE CLANGDWARF_X64_PREFIX = ENV(CLANG_BIN)

> +

> +DEFINE CLANGDWARF_IA32_TARGET = -target i686-pc-linux-gnu

> +DEFINE CLANGDWARF_X64_TARGET = -target x86_64-pc-linux-gnu

> +

> +DEFINE CLANGDWARF_WARNING_OVERRIDES = -Wno-parentheses-

> equality -Wno-tautological-compare -Wno-tautological-constant-out-of-

> range-compare -Wno-empty-body -Wno-unused-const-variable -Wno-

> varargs -Wno-unknown-warning-option -Wno-microsoft-enum-forward-

> reference

> +DEFINE CLANGDWARF_ALL_CC_FLAGS = DEF(GCC48_ALL_CC_FLAGS)

> DEF(CLANGDWARF_WARNING_OVERRIDES) -fno-stack-protector -

> funsigned-char -ftrap-

> function=undefined_behavior_has_been_optimized_away_by_clang -Wno-

> address -Wno-shift-negative-value -Wno-unknown-pragmas -Wno-

> incompatible-library-redeclaration -Wno-null-dereference -mno-implicit-

> float -mms-bitfields -mno-stack-arg-probe -nostdlib -nostdlibinc -fseh-

> exceptions

> +

> +###########################

> +# CLANGDWARF IA32 definitions

> +###########################

> +*_CLANGDWARF_IA32_CC_PATH =