On 12/10/23 11:18, Mike Beaton wrote: > From: Mike Beaton <mjsbea...@gmail.com> > > This warning was already disabled in CLANGDWARF by commit > d3225577123767fd09c91201d27e9c91663ae132. > > gcc can distinguish between optimised-away variable usage (as can occur in > valid debug code) and genuinely unused variables, and only complains about > the latter. clang cannot, and therefore this warning ends up complaining > about valid debug code under clang. > > Since EDK-II code is in general going to be compiled by gcc as well as clang > then disabling this warning in clang does not amount to entirely removing > potentially valid warnings about genuinely unused variables. > > Signed-off-by: Mike Beaton <mjsbea...@gmail.com> > --- > BaseTools/Conf/tools_def.template | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/BaseTools/Conf/tools_def.template > b/BaseTools/Conf/tools_def.template > index c34ecfd557..48cf45245f 100755 > --- a/BaseTools/Conf/tools_def.template > +++ b/BaseTools/Conf/tools_def.template > @@ -1754,7 +1754,7 @@ DEFINE CLANGPDB_X64_PREFIX = ENV(CLANG_BIN) > DEFINE CLANGPDB_IA32_TARGET = -target i686-unknown-windows-gnu > DEFINE CLANGPDB_X64_TARGET = -target x86_64-unknown-windows-gnu > > -DEFINE CLANGPDB_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 -Wno-unaligned-access > -Wno-microsoft-enum-forward-reference > +DEFINE CLANGPDB_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 -Wno-unaligned-access > -Wno-unneeded-internal-declaration -Wno-microsoft-enum-forward-reference > DEFINE CLANGPDB_ALL_CC_FLAGS = DEF(GCC48_ALL_CC_FLAGS) > DEF(CLANGPDB_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 > > ###########################
AFAICT, CLANGPDB_WARNING_OVERRIDES gets included in CLANGPDB_ALL_CC_FLAGS, which in turn gets included in all three of DEBUG, RELEASE and NOOPT build target flags. The original report was "RELEASE CLANGPDB OVMF currently does not compile". Can we use "-Wno-unneeded-internal-declaration" with RELEASE builds only? Thanks, Laszlo -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#112304): https://edk2.groups.io/g/devel/message/112304 Mute This Topic: https://groups.io/mt/103087794/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
