BaseTools: Add mechanism to override or add CC_FLAGS

If environment variable MSFT_EXTRA_CC_FLAGS exists, its contents
are appended to the C compiler flags string for MSFT and INTEL tool
chains. If environment variable GCC_EXTRA_CC_FLAGS exists, its
contents are appended to the C compiler flags string for GCC and
RVCT tool chains.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Scott Duplichan <sc...@notabs.org>
---

This patch must be applied after the gcc49lto. Setting environment
variable GCC_EXTRA_CC_FLAGS=-Wno-error overrides the gcc warnings as
error option so that gcc lto builds can finish and generate a complete
list of warnings. A similar option is added for Microsoft tool chains.
Set environment variable MSFT_EXTRA_CC_FLAGS=/WX- to override warning
as error for Microsoft tool chains. These settings are also useful for
overnight batch building of many EDK2 projects. By overriding warning as
error, the build will continue after a warning is encountered, allowing
more files to be processed. 

 BaseTools/Conf/build_rule.template |  4 ++--
 BaseTools/Conf/tools_def.template  | 30 ++++++++++++++++++------------
 2 files changed, 20 insertions(+), 14 deletions(-)

diff --git a/BaseTools/Conf/build_rule.template 
b/BaseTools/Conf/build_rule.template
index 86ea464..e9802ea 100644
--- a/BaseTools/Conf/build_rule.template
+++ b/BaseTools/Conf/build_rule.template
@@ -128,11 +128,11 @@
         $(OUTPUT_DIR)(+)${s_dir}(+)${s_base}.obj
 
     <Command.MSFT, Command.INTEL>
-        "$(CC)" /Fo${dst} $(CC_FLAGS) $(INC) ${src}
+        "$(CC)" /Fo${dst} $(CC_FLAGS) $(MSFT_EXTRA_CC_FLAGS) $(INC) ${src}
 
     <Command.GCC, Command.RVCT>
         # For RVCTCYGWIN CC_FLAGS must be first to work around pathing issues
-        "$(CC)" $(CC_FLAGS) -o ${dst} $(INC) ${src}
+        "$(CC)" $(CC_FLAGS) $(GCC_EXTRA_CC_FLAGS) -o ${dst} $(INC) ${src}
 
     <Command.ARMGCC, Command.ARMLINUXGCC, command.XCODE>
         "$(CC)" $(CC_FLAGS) -o ${dst} $(INC) ${src}
diff --git a/BaseTools/Conf/tools_def.template 
b/BaseTools/Conf/tools_def.template
index 8d1b319..fa98b66 100644
--- a/BaseTools/Conf/tools_def.template
+++ b/BaseTools/Conf/tools_def.template
@@ -4692,9 +4692,9 @@ RELEASE_GCC49_AARCH64_CC_FLAGS   = 
DEF(GCC49_AARCH64_CC_FLAGS) -Wno-unused-but-s
   DEBUG_GCC49LTO_IA32_CC_FLAGS       = DEF(GCC49LTO_IA32_BASE_CC_FLAGS) 
DEF(GCC49LTO_IA32_DEBUG_FLAGS) -c
 RELEASE_GCC49LTO_IA32_CC_FLAGS       = DEF(GCC49LTO_IA32_BASE_CC_FLAGS) 
DEF(GCC49LTO_IA32_RELEASE_FLAGS) -c
   NOOPT_GCC49LTO_IA32_CC_FLAGS       = DEF(GCC49LTO_IA32_BASE_CC_FLAGS) 
DEF(GCC49LTO_IA32_NOOPT_FLAGS) -c
-  DEBUG_GCC49LTO_IA32_DLINK_FLAGS    = DEF(GCC49LTO_IA32_DLINK_FLAGS) 
DEF(GCC49LTO_IA32_BASE_CC_FLAGS) DEF(GCC49LTO_IA32_DEBUG_FLAGS)
-RELEASE_GCC49LTO_IA32_DLINK_FLAGS    = DEF(GCC49LTO_IA32_DLINK_FLAGS) 
DEF(GCC49LTO_IA32_BASE_CC_FLAGS) DEF(GCC49LTO_IA32_RELEASE_FLAGS)
-  NOOPT_GCC49LTO_IA32_DLINK_FLAGS    = DEF(GCC49LTO_IA32_DLINK_FLAGS) 
DEF(GCC49LTO_IA32_BASE_CC_FLAGS) DEF(GCC49LTO_IA32_NOOPT_FLAGS)
+  DEBUG_GCC49LTO_IA32_DLINK_FLAGS    = DEF(GCC49LTO_IA32_DLINK_FLAGS) 
DEF(GCC49LTO_IA32_BASE_CC_FLAGS) DEF(GCC49LTO_IA32_DEBUG_FLAGS) 
ENV(GCC_EXTRA_CC_FLAGS)
+RELEASE_GCC49LTO_IA32_DLINK_FLAGS    = DEF(GCC49LTO_IA32_DLINK_FLAGS) 
DEF(GCC49LTO_IA32_BASE_CC_FLAGS) DEF(GCC49LTO_IA32_RELEASE_FLAGS) 
ENV(GCC_EXTRA_CC_FLAGS)
+  NOOPT_GCC49LTO_IA32_DLINK_FLAGS    = DEF(GCC49LTO_IA32_DLINK_FLAGS) 
DEF(GCC49LTO_IA32_BASE_CC_FLAGS) DEF(GCC49LTO_IA32_NOOPT_FLAGS) 
ENV(GCC_EXTRA_CC_FLAGS)
       *_GCC49LTO_IA32_RC_FLAGS       = DEF(GCC_IA32_RC_FLAGS)
       *_GCC49LTO_IA32_OBJCOPY_FLAGS  =
       *_GCC49LTO_IA32_NASM_FLAGS     = -f elf32
@@ -4720,9 +4720,9 @@ RELEASE_GCC49LTO_IA32_DLINK_FLAGS    = 
DEF(GCC49LTO_IA32_DLINK_FLAGS) DEF(GCC49L
   DEBUG_GCC49LTO_X64_CC_FLAGS        = DEF(GCC49LTO_X64_BASE_CC_FLAGS) 
DEF(GCC49LTO_X64_DEBUG_FLAGS) -c
 RELEASE_GCC49LTO_X64_CC_FLAGS        = DEF(GCC49LTO_X64_BASE_CC_FLAGS) 
DEF(GCC49LTO_X64_RELEASE_FLAGS) -c
   NOOPT_GCC49LTO_X64_CC_FLAGS        = DEF(GCC49LTO_X64_BASE_CC_FLAGS) 
DEF(GCC49LTO_X64_NOOPT_FLAGS) -c
-  DEBUG_GCC49LTO_X64_DLINK_FLAGS     = DEF(GCC49LTO_X64_DLINK_FLAGS) 
DEF(GCC49LTO_X64_BASE_CC_FLAGS) DEF(GCC49LTO_X64_DEBUG_FLAGS)
-RELEASE_GCC49LTO_X64_DLINK_FLAGS     = DEF(GCC49LTO_X64_DLINK_FLAGS) 
DEF(GCC49LTO_X64_BASE_CC_FLAGS) DEF(GCC49LTO_X64_RELEASE_FLAGS)
-  NOOPT_GCC49LTO_X64_DLINK_FLAGS     = DEF(GCC49LTO_X64_DLINK_FLAGS) 
DEF(GCC49LTO_X64_BASE_CC_FLAGS) DEF(GCC49LTO_X64_NOOPT_FLAGS)
+  DEBUG_GCC49LTO_X64_DLINK_FLAGS     = DEF(GCC49LTO_X64_DLINK_FLAGS) 
DEF(GCC49LTO_X64_BASE_CC_FLAGS) DEF(GCC49LTO_X64_DEBUG_FLAGS) 
ENV(GCC_EXTRA_CC_FLAGS)
+RELEASE_GCC49LTO_X64_DLINK_FLAGS     = DEF(GCC49LTO_X64_DLINK_FLAGS) 
DEF(GCC49LTO_X64_BASE_CC_FLAGS) DEF(GCC49LTO_X64_RELEASE_FLAGS) 
ENV(GCC_EXTRA_CC_FLAGS)
+  NOOPT_GCC49LTO_X64_DLINK_FLAGS     = DEF(GCC49LTO_X64_DLINK_FLAGS) 
DEF(GCC49LTO_X64_BASE_CC_FLAGS) DEF(GCC49LTO_X64_NOOPT_FLAGS) 
ENV(GCC_EXTRA_CC_FLAGS)
       *_GCC49LTO_X64_RC_FLAGS        = DEF(GCC_X64_RC_FLAGS)
       *_GCC49LTO_X64_OBJCOPY_FLAGS   =
       *_GCC49LTO_X64_NASM_FLAGS      = -f elf64
@@ -4756,9 +4756,9 @@ RELEASE_GCC49LTO_X64_DLINK_FLAGS     = 
DEF(GCC49LTO_X64_DLINK_FLAGS) DEF(GCC49LT
   DEBUG_GCC49LTO_ARM_CC_FLAGS        = DEF(GCC49LTO_ARM_BASE_CC_FLAGS) 
DEF(GCC49LTO_ARM_DEBUG_FLAGS) -c
 RELEASE_GCC49LTO_ARM_CC_FLAGS        = DEF(GCC49LTO_ARM_BASE_CC_FLAGS) 
DEF(GCC49LTO_ARM_RELEASE_FLAGS) -c
   NOOPT_GCC49LTO_ARM_CC_FLAGS        = DEF(GCC49LTO_ARM_BASE_CC_FLAGS) 
DEF(GCC49LTO_ARM_NOOPT_FLAGS) -c
-  DEBUG_GCC49LTO_ARM_DLINK_FLAGS     = DEF(GCC49LTO_ARM_DLINK_FLAGS) 
DEF(GCC49LTO_ARM_BASE_CC_FLAGS) DEF(GCC49LTO_ARM_DEBUG_FLAGS)
-RELEASE_GCC49LTO_ARM_DLINK_FLAGS     = DEF(GCC49LTO_ARM_DLINK_FLAGS) 
DEF(GCC49LTO_ARM_BASE_CC_FLAGS) DEF(GCC49LTO_ARM_RELEASE_FLAGS)
-  NOOPT_GCC49LTO_ARM_DLINK_FLAGS     = DEF(GCC49LTO_ARM_DLINK_FLAGS) 
DEF(GCC49LTO_ARM_BASE_CC_FLAGS) DEF(GCC49LTO_ARM_NOOPT_FLAGS)
+  DEBUG_GCC49LTO_ARM_DLINK_FLAGS     = DEF(GCC49LTO_ARM_DLINK_FLAGS) 
DEF(GCC49LTO_ARM_BASE_CC_FLAGS) DEF(GCC49LTO_ARM_DEBUG_FLAGS) 
ENV(GCC_EXTRA_CC_FLAGS)
+RELEASE_GCC49LTO_ARM_DLINK_FLAGS     = DEF(GCC49LTO_ARM_DLINK_FLAGS) 
DEF(GCC49LTO_ARM_BASE_CC_FLAGS) DEF(GCC49LTO_ARM_RELEASE_FLAGS) 
ENV(GCC_EXTRA_CC_FLAGS)
+  NOOPT_GCC49LTO_ARM_DLINK_FLAGS     = DEF(GCC49LTO_ARM_DLINK_FLAGS) 
DEF(GCC49LTO_ARM_BASE_CC_FLAGS) DEF(GCC49LTO_ARM_NOOPT_FLAGS) 
ENV(GCC_EXTRA_CC_FLAGS)
 
 ##################
 # GCC49LTO AARCH64 definitions
@@ -4785,9 +4785,9 @@ RELEASE_GCC49LTO_ARM_DLINK_FLAGS     = 
DEF(GCC49LTO_ARM_DLINK_FLAGS) DEF(GCC49LT
   DEBUG_GCC49LTO_AARCH64_CC_FLAGS    = DEF(GCC49LTO_AARCH64_BASE_CC_FLAGS) 
DEF(GCC49LTO_AARCH64_DEBUG_FLAGS) -c
 RELEASE_GCC49LTO_AARCH64_CC_FLAGS    = DEF(GCC49LTO_AARCH64_BASE_CC_FLAGS) 
DEF(GCC49LTO_AARCH64_RELEASE_FLAGS) -c
   NOOPT_GCC49LTO_AARCH64_CC_FLAGS    = DEF(GCC49LTO_AARCH64_BASE_CC_FLAGS) 
DEF(GCC49LTO_AARCH64_NOOPT_FLAGS) -c
-  DEBUG_GCC49LTO_AARCH64_DLINK_FLAGS = DEF(GCC49LTO_AARCH64_DLINK_FLAGS) 
DEF(GCC49LTO_AARCH64_BASE_CC_FLAGS) DEF(GCC49LTO_AARCH64_DEBUG_FLAGS)
-RELEASE_GCC49LTO_AARCH64_DLINK_FLAGS = DEF(GCC49LTO_AARCH64_DLINK_FLAGS) 
DEF(GCC49LTO_AARCH64_BASE_CC_FLAGS) DEF(GCC49LTO_AARCH64_RELEASE_FLAGS)
-  NOOPT_GCC49LTO_AARCH64_DLINK_FLAGS = DEF(GCC49LTO_AARCH64_DLINK_FLAGS) 
DEF(GCC49LTO_AARCH64_BASE_CC_FLAGS) DEF(GCC49LTO_AARCH64_NOOPT_FLAGS)
+  DEBUG_GCC49LTO_AARCH64_DLINK_FLAGS = DEF(GCC49LTO_AARCH64_DLINK_FLAGS) 
DEF(GCC49LTO_AARCH64_BASE_CC_FLAGS) DEF(GCC49LTO_AARCH64_DEBUG_FLAGS) 
ENV(GCC_EXTRA_CC_FLAGS)
+RELEASE_GCC49LTO_AARCH64_DLINK_FLAGS = DEF(GCC49LTO_AARCH64_DLINK_FLAGS) 
DEF(GCC49LTO_AARCH64_BASE_CC_FLAGS) DEF(GCC49LTO_AARCH64_RELEASE_FLAGS) 
ENV(GCC_EXTRA_CC_FLAGS)
+  NOOPT_GCC49LTO_AARCH64_DLINK_FLAGS = DEF(GCC49LTO_AARCH64_DLINK_FLAGS) 
DEF(GCC49LTO_AARCH64_BASE_CC_FLAGS) DEF(GCC49LTO_AARCH64_NOOPT_FLAGS) 
ENV(GCC_EXTRA_CC_FLAGS)
 
 
####################################################################################
 #
@@ -7129,3 +7129,9 @@ RELEASE_ARMLINUXGCC_AARCH64_CC_FLAGS = $(ARCHCC_FLAGS) 
$(PLATFORM_FLAGS) DEF(GCC
 # Build rule order
 #################
 *_*_*_*_BUILDRULEORDER = nasm asm Asm ASM S s
+
+#################
+# Extra C compiler flags
+#################
+*_*_*_*_MSFT_EXTRA_CC_FLAGS = ENV(MSFT_EXTRA_CC_FLAGS)
+*_*_*_*_GCC_EXTRA_CC_FLAGS = ENV(GCC_EXTRA_CC_FLAGS)


------------------------------------------------------------------------------
Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that
you need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to