diff options
author | Mike Beaton <mjsbeaton@gmail.com> | 2024-09-29 11:40:44 +0100 |
---|---|---|
committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2024-10-18 16:22:07 +0000 |
commit | 8f84eb0e761992dc1218f63ee53a54ac0c65630f (patch) | |
tree | 462e50b15469dd7bd1c6854b44036ae7852ef172 /BaseTools/Scripts | |
parent | ae83c6b7fd83a5906e016a32027c1bcd792a624e (diff) | |
download | edk2-8f84eb0e761992dc1218f63ee53a54ac0c65630f.tar.gz edk2-8f84eb0e761992dc1218f63ee53a54ac0c65630f.tar.bz2 edk2-8f84eb0e761992dc1218f63ee53a54ac0c65630f.zip |
BaseTools: Remove -Wno-unneeded-internal-declaration from CLANGDWARF
Subsequent to updating the 'null' DEBUG macro to explicitly
discard its expression, it is possible to remove this warning
suppression from CLANGDWARF and still successfully compile
its RELEASE build.
Note that CLANGPDB did and does not have this warning suppressed,
and so before updating the 'null' DEBUG macro, CLANGPDB RELEASE
was not building successfully in recent versions of clang,
but was stopping with the error:
.../edk2/OvmfPkg/VirtioSerialDxe/VirtioSerial.c:28:22: error:
variable 'EventNames' is not needed and will not be
emitted [-Werror,-Wunneeded-internal-declaration]
STATIC CONST CHAR8 *EventNames[] = {
^
This change makes the two CLANG variants match with respect
to this warning, and leaves the warning enabled which is
considered a benefit as it has the potential to catch real
coding errors
Signed-off-by: Mike Beaton <mjsbeaton@gmail.com>
Diffstat (limited to 'BaseTools/Scripts')
0 files changed, 0 insertions, 0 deletions