summaryrefslogtreecommitdiffstats
path: root/BaseTools/Scripts
diff options
context:
space:
mode:
authorMike Beaton <mjsbeaton@gmail.com>2024-09-29 11:40:44 +0100
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>2024-10-18 16:22:07 +0000
commit8f84eb0e761992dc1218f63ee53a54ac0c65630f (patch)
tree462e50b15469dd7bd1c6854b44036ae7852ef172 /BaseTools/Scripts
parentae83c6b7fd83a5906e016a32027c1bcd792a624e (diff)
downloadedk2-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 &apos;EventNames&apos; 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