Reviewed-by: Michael D Kinney <michael.d.kin...@intel.com> Mike
> -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Rebecca Cran > Sent: Friday, February 4, 2022 11:51 AM > To: devel@edk2.groups.io; Feng, Bob C <bob.c.f...@intel.com>; Gao, Liming > <gaolim...@byosoft.com.cn>; Chen, Christine > <yuwei.c...@intel.com>; Kinney, Michael D <michael.d.kin...@intel.com>; > Andrew Fish <af...@apple.com> > Cc: Rebecca Cran <quic_rc...@quicinc.com> > Subject: [edk2-devel] [PATCH 1/1] BaseTools: Update > CLANG{35,38}_WARNING_OVERRIDES to ignore unused vars > > Building with the CLANG35 and CLANG38 toolset fails because of variables > which are set but not otherwise used in the RELEASE build. > > GCC added -Wno-unused-but-set-variable back in 2016, and later added > -Wno-unused-const-variable. Add those to CLANG35_WARNING_OVERRIDES and > CLANG38_WARNING_OVERRIDES. > > Signed-off-by: Rebecca Cran <quic_rc...@quicinc.com> > --- > BaseTools/Conf/tools_def.template | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/BaseTools/Conf/tools_def.template > b/BaseTools/Conf/tools_def.template > index 2e6b382ab623..e839e8602a0b 100755 > --- a/BaseTools/Conf/tools_def.template > +++ b/BaseTools/Conf/tools_def.template > @@ -2496,7 +2496,7 @@ RELEASE_GCC5_AARCH64_DLINK_XIPFLAGS = -z > common-page-size=0x20 > DEFINE CLANG35_ARM_TARGET = -target arm-linux-gnueabi > DEFINE CLANG35_AARCH64_TARGET = -target aarch64-linux-gnu > > -DEFINE CLANG35_WARNING_OVERRIDES = -Wno-parentheses-equality > -Wno-tautological-compare -Wno-tautological-constant-out-of- > range-compare -Wno-empty-body -Wno-unknown-warning-option > +DEFINE CLANG35_WARNING_OVERRIDES = -Wno-parentheses-equality > -Wno-tautological-compare -Wno-tautological-constant-out-of- > range-compare -Wno-empty-body -Wno-unknown-warning-option > -Wno-unused-but-set-variable -Wno-unused-const-variable > DEFINE CLANG35_ARM_CC_FLAGS = DEF(GCC_ARM_CC_FLAGS) > DEF(CLANG35_ARM_TARGET) DEF(CLANG35_WARNING_OVERRIDES) > DEFINE CLANG35_AARCH64_CC_FLAGS = DEF(GCC_AARCH64_CC_FLAGS) > DEF(CLANG35_AARCH64_TARGET) -mcmodel=small > DEF(CLANG35_WARNING_OVERRIDES) > > @@ -2566,7 +2566,7 @@ DEFINE CLANG38_X64_PREFIX = ENV(CLANG38_BIN) > DEFINE CLANG38_IA32_TARGET = -target i686-pc-linux-gnu > DEFINE CLANG38_X64_TARGET = -target x86_64-pc-linux-gnu > > -DEFINE CLANG38_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 > +DEFINE CLANG38_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-unused-but-set-variable > -Wno-unused-const-variable > DEFINE CLANG38_ALL_CC_FLAGS = DEF(GCC48_ALL_CC_FLAGS) > DEF(CLANG38_WARNING_OVERRIDES) -fno-stack-protector -mms- > bitfields -Wno-address -Wno-shift-negative-value -Wno-unknown-pragmas > -Wno-incompatible-library-redeclaration -fno- > asynchronous-unwind-tables -mno-sse -mno-mmx -msoft-float -mno-implicit-float > -ftrap- > function=undefined_behavior_has_been_optimized_away_by_clang -funsigned-char > -fno-ms-extensions -Wno-null-dereference > > ########################### > -- > 2.34.1 > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#86393): https://edk2.groups.io/g/devel/message/86393 Mute This Topic: https://groups.io/mt/88915148/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-