Bruce,

On 09/30/16 09:44, Yonghong Zhu wrote:
> Update the tools_def.template to add NOOPT support with GCC tool chains.
> 
> Cc: Liming Gao <[email protected]>
> Cc: Laszlo Ersek <[email protected]>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Yonghong Zhu <[email protected]>
> ---
>  BaseTools/Conf/tools_def.template | 27 +++++++++++++++++++++++++++
>  1 file changed, 27 insertions(+)

Can you please test this in the environment where you wanted to use it
originally? (I assume you were using some kind of source level debugging.)

For that, you'll probably have to add NOOPT to BUILD_TARGETS in the
OvmfPkg/OvmfPkg*.dsc files. If you do that and everything works fine for
you, I'd appreciate if you could also submit the OvmfPkg patch to the list.

Thanks!
Laszlo

> diff --git a/BaseTools/Conf/tools_def.template 
> b/BaseTools/Conf/tools_def.template
> index 5414454..54fccec 100755
> --- a/BaseTools/Conf/tools_def.template
> +++ b/BaseTools/Conf/tools_def.template
> @@ -4767,10 +4767,11 @@ DEFINE GCC5_AARCH64_ASLDLINK_FLAGS   = 
> DEF(GCC49_AARCH64_ASLDLINK_FLAGS)
>  *_GCC46_IA32_OBJCOPY_FLAGS        = 
>  *_GCC46_IA32_NASM_FLAGS           = -f elf32
>  
>    DEBUG_GCC46_IA32_CC_FLAGS       = DEF(GCC46_IA32_CC_FLAGS) -Os
>  RELEASE_GCC46_IA32_CC_FLAGS       = DEF(GCC46_IA32_CC_FLAGS) -Os 
> -Wno-unused-but-set-variable
> +  NOOPT_GCC46_IA32_CC_FLAGS       = DEF(GCC46_IA32_CC_FLAGS) -O0
>  
>  ##################
>  # GCC46 X64 definitions
>  ##################
>  *_GCC46_X64_OBJCOPY_PATH         = DEF(GCC46_X64_PREFIX)objcopy
> @@ -4794,10 +4795,11 @@ RELEASE_GCC46_IA32_CC_FLAGS       = 
> DEF(GCC46_IA32_CC_FLAGS) -Os -Wno-unused-but
>  *_GCC46_X64_OBJCOPY_FLAGS        = 
>  *_GCC46_X64_NASM_FLAGS           = -f elf64
>  
>    DEBUG_GCC46_X64_CC_FLAGS       = DEF(GCC46_X64_CC_FLAGS)
>  RELEASE_GCC46_X64_CC_FLAGS       = DEF(GCC46_X64_CC_FLAGS) 
> -Wno-unused-but-set-variable
> +  NOOPT_GCC46_X64_CC_FLAGS       = DEF(GCC46_X64_CC_FLAGS) -O0
>  
>  ##################
>  # GCC46 ARM definitions
>  ##################
>  *_GCC46_ARM_OBJCOPY_PATH         = echo
> @@ -4826,10 +4828,11 @@ RELEASE_GCC46_X64_CC_FLAGS       = 
> DEF(GCC46_X64_CC_FLAGS) -Wno-unused-but-set-v
>  *_GCC46_ARM_VFRPP_FLAGS          = $(ARCHCC_FLAGS) $(PLATFORM_FLAGS) 
> DEF(GCC_VFRPP_FLAGS)
>  *_GCC46_ARM_CC_XIPFLAGS          = DEF(GCC46_ARM_CC_XIPFLAGS)
>  
>    DEBUG_GCC46_ARM_CC_FLAGS       = DEF(GCC46_ARM_CC_FLAGS) -O0
>  RELEASE_GCC46_ARM_CC_FLAGS       = DEF(GCC46_ARM_CC_FLAGS) 
> -Wno-unused-but-set-variable
> +  NOOPT_GCC46_ARM_CC_FLAGS       = DEF(GCC46_ARM_CC_FLAGS) -O0
>  
>  
> ####################################################################################
>  #
>  # GCC 4.7 - This configuration is used to compile under Linux to produce
>  #           PE/COFF binaries using GCC 4.7.
> @@ -4873,10 +4876,11 @@ RELEASE_GCC46_ARM_CC_FLAGS       = 
> DEF(GCC46_ARM_CC_FLAGS) -Wno-unused-but-set-v
>  *_GCC47_IA32_OBJCOPY_FLAGS        = 
>  *_GCC47_IA32_NASM_FLAGS           = -f elf32
>  
>    DEBUG_GCC47_IA32_CC_FLAGS       = DEF(GCC47_IA32_CC_FLAGS) -Os
>  RELEASE_GCC47_IA32_CC_FLAGS       = DEF(GCC47_IA32_CC_FLAGS) -Os 
> -Wno-unused-but-set-variable
> +  NOOPT_GCC47_IA32_CC_FLAGS       = DEF(GCC47_IA32_CC_FLAGS) -O0
>  
>  ##################
>  # GCC47 X64 definitions
>  ##################
>  *_GCC47_X64_OBJCOPY_PATH         = DEF(GCC47_X64_PREFIX)objcopy
> @@ -4900,10 +4904,11 @@ RELEASE_GCC47_IA32_CC_FLAGS       = 
> DEF(GCC47_IA32_CC_FLAGS) -Os -Wno-unused-but
>  *_GCC47_X64_OBJCOPY_FLAGS        = 
>  *_GCC47_X64_NASM_FLAGS           = -f elf64
>  
>    DEBUG_GCC47_X64_CC_FLAGS       = DEF(GCC47_X64_CC_FLAGS)
>  RELEASE_GCC47_X64_CC_FLAGS       = DEF(GCC47_X64_CC_FLAGS) 
> -Wno-unused-but-set-variable
> +  NOOPT_GCC47_X64_CC_FLAGS       = DEF(GCC47_X64_CC_FLAGS) -O0
>  
>  ##################
>  # GCC47 ARM definitions
>  ##################
>  *_GCC47_ARM_CC_PATH              = ENV(GCC47_ARM_PREFIX)gcc
> @@ -4931,10 +4936,11 @@ RELEASE_GCC47_X64_CC_FLAGS       = 
> DEF(GCC47_X64_CC_FLAGS) -Wno-unused-but-set-v
>  *_GCC47_ARM_VFRPP_FLAGS          = $(ARCHCC_FLAGS) $(PLATFORM_FLAGS) 
> DEF(GCC_VFRPP_FLAGS)
>  *_GCC47_ARM_CC_XIPFLAGS          = DEF(GCC47_ARM_CC_XIPFLAGS)
>  
>    DEBUG_GCC47_ARM_CC_FLAGS       = DEF(GCC47_ARM_CC_FLAGS) -O0
>  RELEASE_GCC47_ARM_CC_FLAGS       = DEF(GCC47_ARM_CC_FLAGS) 
> -Wno-unused-but-set-variable
> +  NOOPT_GCC47_ARM_CC_FLAGS       = DEF(GCC47_ARM_CC_FLAGS) -O0
>  
>  ##################
>  # GCC47 AARCH64 definitions
>  ##################
>  *_GCC47_AARCH64_CC_PATH          = ENV(GCC47_AARCH64_PREFIX)gcc
> @@ -4959,10 +4965,11 @@ RELEASE_GCC47_ARM_CC_FLAGS       = 
> DEF(GCC47_ARM_CC_FLAGS) -Wno-unused-but-set-v
>  *_GCC47_AARCH64_VFRPP_FLAGS      = $(ARCHCC_FLAGS) $(PLATFORM_FLAGS) 
> DEF(GCC_VFRPP_FLAGS)
>  *_GCC47_AARCH64_CC_XIPFLAGS      = DEF(GCC47_AARCH64_CC_XIPFLAGS)
>  
>    DEBUG_GCC47_AARCH64_CC_FLAGS   = DEF(GCC47_AARCH64_CC_FLAGS) -O0
>  RELEASE_GCC47_AARCH64_CC_FLAGS   = DEF(GCC47_AARCH64_CC_FLAGS) 
> -Wno-unused-but-set-variable -fomit-frame-pointer
> +  NOOPT_GCC47_AARCH64_CC_FLAGS   = DEF(GCC47_AARCH64_CC_FLAGS) -O0
>  
>  
> ####################################################################################
>  #
>  # GCC 4.8 - This configuration is used to compile under Linux to produce
>  #           PE/COFF binaries using GCC 4.8.
> @@ -5006,10 +5013,11 @@ RELEASE_GCC47_AARCH64_CC_FLAGS   = 
> DEF(GCC47_AARCH64_CC_FLAGS) -Wno-unused-but-s
>  *_GCC48_IA32_OBJCOPY_FLAGS        = 
>  *_GCC48_IA32_NASM_FLAGS           = -f elf32
>  
>    DEBUG_GCC48_IA32_CC_FLAGS       = DEF(GCC48_IA32_CC_FLAGS) -Os
>  RELEASE_GCC48_IA32_CC_FLAGS       = DEF(GCC48_IA32_CC_FLAGS) -Os 
> -Wno-unused-but-set-variable
> +  NOOPT_GCC48_IA32_CC_FLAGS       = DEF(GCC48_IA32_CC_FLAGS) -O0
>  
>  ##################
>  # GCC48 X64 definitions
>  ##################
>  *_GCC48_X64_OBJCOPY_PATH         = DEF(GCC48_X64_PREFIX)objcopy
> @@ -5033,10 +5041,11 @@ RELEASE_GCC48_IA32_CC_FLAGS       = 
> DEF(GCC48_IA32_CC_FLAGS) -Os -Wno-unused-but
>  *_GCC48_X64_OBJCOPY_FLAGS        = 
>  *_GCC48_X64_NASM_FLAGS           = -f elf64
>  
>    DEBUG_GCC48_X64_CC_FLAGS       = DEF(GCC48_X64_CC_FLAGS)
>  RELEASE_GCC48_X64_CC_FLAGS       = DEF(GCC48_X64_CC_FLAGS) 
> -Wno-unused-but-set-variable
> +  NOOPT_GCC48_X64_CC_FLAGS       = DEF(GCC48_X64_CC_FLAGS) -O0
>  
>  ##################
>  # GCC48 ARM definitions
>  ##################
>  *_GCC48_ARM_CC_PATH              = ENV(GCC48_ARM_PREFIX)gcc
> @@ -5064,10 +5073,11 @@ RELEASE_GCC48_X64_CC_FLAGS       = 
> DEF(GCC48_X64_CC_FLAGS) -Wno-unused-but-set-v
>  *_GCC48_ARM_VFRPP_FLAGS          = $(ARCHCC_FLAGS) $(PLATFORM_FLAGS) 
> DEF(GCC_VFRPP_FLAGS)
>  *_GCC48_ARM_CC_XIPFLAGS          = DEF(GCC48_ARM_CC_XIPFLAGS)
>  
>    DEBUG_GCC48_ARM_CC_FLAGS       = DEF(GCC48_ARM_CC_FLAGS) -O0
>  RELEASE_GCC48_ARM_CC_FLAGS       = DEF(GCC48_ARM_CC_FLAGS) 
> -Wno-unused-but-set-variable
> +  NOOPT_GCC48_ARM_CC_FLAGS       = DEF(GCC48_ARM_CC_FLAGS) -O0
>  
>  ##################
>  # GCC48 AARCH64 definitions
>  ##################
>  *_GCC48_AARCH64_CC_PATH          = ENV(GCC48_AARCH64_PREFIX)gcc
> @@ -5092,10 +5102,11 @@ RELEASE_GCC48_ARM_CC_FLAGS       = 
> DEF(GCC48_ARM_CC_FLAGS) -Wno-unused-but-set-v
>  *_GCC48_AARCH64_VFRPP_FLAGS      = $(ARCHCC_FLAGS) $(PLATFORM_FLAGS) 
> DEF(GCC_VFRPP_FLAGS)
>  *_GCC48_AARCH64_CC_XIPFLAGS      = DEF(GCC48_AARCH64_CC_XIPFLAGS)
>  
>    DEBUG_GCC48_AARCH64_CC_FLAGS   = DEF(GCC48_AARCH64_CC_FLAGS) -O0
>  RELEASE_GCC48_AARCH64_CC_FLAGS   = DEF(GCC48_AARCH64_CC_FLAGS) 
> -Wno-unused-but-set-variable -fomit-frame-pointer
> +  NOOPT_GCC48_AARCH64_CC_FLAGS   = DEF(GCC48_AARCH64_CC_FLAGS) -O0
>  
>  
> ####################################################################################
>  #
>  # GCC 4.9 - This configuration is used to compile under Linux to produce
>  #           PE/COFF binaries using GCC 4.9.
> @@ -5139,10 +5150,11 @@ RELEASE_GCC48_AARCH64_CC_FLAGS   = 
> DEF(GCC48_AARCH64_CC_FLAGS) -Wno-unused-but-s
>  *_GCC49_IA32_OBJCOPY_FLAGS        = 
>  *_GCC49_IA32_NASM_FLAGS           = -f elf32
>  
>    DEBUG_GCC49_IA32_CC_FLAGS       = DEF(GCC49_IA32_CC_FLAGS) -Os
>  RELEASE_GCC49_IA32_CC_FLAGS       = DEF(GCC49_IA32_CC_FLAGS) -Os 
> -Wno-unused-but-set-variable
> +  NOOPT_GCC49_IA32_CC_FLAGS       = DEF(GCC49_IA32_CC_FLAGS) -O0
>  
>  ##################
>  # GCC49 X64 definitions
>  ##################
>  *_GCC49_X64_OBJCOPY_PATH         = DEF(GCC49_X64_PREFIX)objcopy
> @@ -5166,10 +5178,11 @@ RELEASE_GCC49_IA32_CC_FLAGS       = 
> DEF(GCC49_IA32_CC_FLAGS) -Os -Wno-unused-but
>  *_GCC49_X64_OBJCOPY_FLAGS        = 
>  *_GCC49_X64_NASM_FLAGS           = -f elf64
>  
>    DEBUG_GCC49_X64_CC_FLAGS       = DEF(GCC49_X64_CC_FLAGS)
>  RELEASE_GCC49_X64_CC_FLAGS       = DEF(GCC49_X64_CC_FLAGS) 
> -Wno-unused-but-set-variable
> +  NOOPT_GCC49_X64_CC_FLAGS       = DEF(GCC49_X64_CC_FLAGS) -O0
>  
>  ##################
>  # GCC49 ARM definitions
>  ##################
>  *_GCC49_ARM_CC_PATH              = ENV(GCC49_ARM_PREFIX)gcc
> @@ -5197,10 +5210,11 @@ RELEASE_GCC49_X64_CC_FLAGS       = 
> DEF(GCC49_X64_CC_FLAGS) -Wno-unused-but-set-v
>  *_GCC49_ARM_VFRPP_FLAGS          = $(ARCHCC_FLAGS) $(PLATFORM_FLAGS) 
> DEF(GCC_VFRPP_FLAGS)
>  *_GCC49_ARM_CC_XIPFLAGS          = DEF(GCC49_ARM_CC_XIPFLAGS)
>  
>    DEBUG_GCC49_ARM_CC_FLAGS       = DEF(GCC49_ARM_CC_FLAGS) -O0
>  RELEASE_GCC49_ARM_CC_FLAGS       = DEF(GCC49_ARM_CC_FLAGS) 
> -Wno-unused-but-set-variable
> +  NOOPT_GCC49_ARM_CC_FLAGS       = DEF(GCC49_ARM_CC_FLAGS) -O0
>  
>  ##################
>  # GCC49 AARCH64 definitions
>  ##################
>  *_GCC49_AARCH64_CC_PATH          = ENV(GCC49_AARCH64_PREFIX)gcc
> @@ -5229,10 +5243,14 @@ RELEASE_GCC49_ARM_CC_FLAGS       = 
> DEF(GCC49_ARM_CC_FLAGS) -Wno-unused-but-set-v
>    DEBUG_GCC49_AARCH64_DLINK_XIPFLAGS = -z common-page-size=0x20
>  
>  RELEASE_GCC49_AARCH64_CC_FLAGS     = DEF(GCC49_AARCH64_CC_FLAGS) 
> -Wno-unused-but-set-variable -mcmodel=tiny -fomit-frame-pointer
>  RELEASE_GCC49_AARCH64_DLINK_FLAGS  = DEF(GCC49_AARCH64_DLINK_FLAGS)
>  
> +  NOOPT_GCC49_AARCH64_CC_FLAGS     = DEF(GCC49_AARCH64_CC_FLAGS) -O0 
> -mcmodel=small
> +  NOOPT_GCC49_AARCH64_DLINK_FLAGS  = DEF(GCC49_AARCH64_DLINK_FLAGS) -z 
> common-page-size=0x1000 -O0
> +  NOOPT_GCC49_AARCH64_DLINK_XIPFLAGS = -z common-page-size=0x20 -O0
> +
>  
> ####################################################################################
>  #
>  # GCC 5 - This configuration is used to compile under Linux to produce
>  #         PE/COFF binaries using GCC 5
>  #
> @@ -5275,10 +5293,11 @@ RELEASE_GCC49_AARCH64_DLINK_FLAGS  = 
> DEF(GCC49_AARCH64_DLINK_FLAGS)
>  *_GCC5_IA32_OBJCOPY_FLAGS        =
>  *_GCC5_IA32_NASM_FLAGS           = -f elf32
>  
>    DEBUG_GCC5_IA32_CC_FLAGS       = DEF(GCC5_IA32_CC_FLAGS) -Os
>  RELEASE_GCC5_IA32_CC_FLAGS       = DEF(GCC5_IA32_CC_FLAGS) -Os 
> -Wno-unused-but-set-variable
> +  NOOPT_GCC5_IA32_CC_FLAGS       = DEF(GCC5_IA32_CC_FLAGS) -O0
>  
>  ##################
>  # GCC5 X64 definitions
>  ##################
>  *_GCC5_X64_OBJCOPY_PATH          = DEF(GCC5_X64_PREFIX)objcopy
> @@ -5302,10 +5321,11 @@ RELEASE_GCC5_IA32_CC_FLAGS       = 
> DEF(GCC5_IA32_CC_FLAGS) -Os -Wno-unused-but-s
>  *_GCC5_X64_OBJCOPY_FLAGS         =
>  *_GCC5_X64_NASM_FLAGS            = -f elf64
>  
>    DEBUG_GCC5_X64_CC_FLAGS        = DEF(GCC5_X64_CC_FLAGS)
>  RELEASE_GCC5_X64_CC_FLAGS        = DEF(GCC5_X64_CC_FLAGS) 
> -Wno-unused-but-set-variable
> +  NOOPT_GCC5_X64_CC_FLAGS        = DEF(GCC5_X64_CC_FLAGS) -O0
>  
>  ##################
>  # GCC5 ARM definitions
>  ##################
>  *_GCC5_ARM_CC_PATH               = ENV(GCC5_ARM_PREFIX)gcc
> @@ -5334,10 +5354,13 @@ RELEASE_GCC5_X64_CC_FLAGS        = 
> DEF(GCC5_X64_CC_FLAGS) -Wno-unused-but-set-va
>    DEBUG_GCC5_ARM_DLINK_FLAGS     = DEF(GCC5_ARM_DLINK_FLAGS)
>  
>  RELEASE_GCC5_ARM_CC_FLAGS        = DEF(GCC5_ARM_CC_FLAGS) -flto 
> -Wno-unused-but-set-variable
>  RELEASE_GCC5_ARM_DLINK_FLAGS     = DEF(GCC5_ARM_DLINK_FLAGS) -flto -Os 
> -L$(WORKSPACE)/ArmPkg/Library/GccLto -llto-arm 
> -Wl,-plugin-opt=-pass-through=-llto-arm
>  
> +  NOOPT_GCC5_ARM_CC_FLAGS        = DEF(GCC5_ARM_CC_FLAGS) -O0
> +  NOOPT_GCC5_ARM_DLINK_FLAGS     = DEF(GCC5_ARM_DLINK_FLAGS) -O0
> +
>  ##################
>  # GCC5 AARCH64 definitions
>  ##################
>  *_GCC5_AARCH64_CC_PATH           = ENV(GCC5_AARCH64_PREFIX)gcc
>  *_GCC5_AARCH64_SLINK_PATH        = ENV(GCC5_AARCH64_PREFIX)gcc-ar
> @@ -5365,10 +5388,14 @@ RELEASE_GCC5_ARM_DLINK_FLAGS     = 
> DEF(GCC5_ARM_DLINK_FLAGS) -flto -Os -L$(WORKS
>    DEBUG_GCC5_AARCH64_DLINK_XIPFLAGS = -z common-page-size=0x20
>  
>  RELEASE_GCC5_AARCH64_CC_FLAGS    = DEF(GCC5_AARCH64_CC_FLAGS) -flto 
> -Wno-unused-but-set-variable -mcmodel=tiny -fomit-frame-pointer
>  RELEASE_GCC5_AARCH64_DLINK_FLAGS = DEF(GCC5_AARCH64_DLINK_FLAGS) -flto -Os 
> -L$(WORKSPACE)/ArmPkg/Library/GccLto -llto-aarch64 
> -Wl,-plugin-opt=-pass-through=-llto-aarch64
>  
> +  NOOPT_GCC5_AARCH64_CC_FLAGS    = DEF(GCC5_AARCH64_CC_FLAGS) -O0 
> -mcmodel=small
> +  NOOPT_GCC5_AARCH64_DLINK_FLAGS = DEF(GCC5_AARCH64_DLINK_FLAGS) -z 
> common-page-size=0x1000 -O0
> +  NOOPT_GCC5_AARCH64_DLINK_XIPFLAGS = -z common-page-size=0x20 -O0
> +
>  
> ####################################################################################
>  #
>  # CLANG35   - This configuration is used to compile under Linux to produce
>  #             PE/COFF binaries using the clang compiler and assembler (v3.5 
> and up)
>  #             and GNU linker
> 

_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to