From 15ca5ce99a80d9ebb5ef2b1aca6ea00784931de4 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Kai=20Wasserb=C3=A4ch?= Date: Fri, 17 Aug 2018 12:14:22 +0200 Subject: [PATCH] amd/addrlib: mark returnCode as MAYBE_UNUSED in ElemGetExportNorm MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Only used, when asserts are enabled. Fixes an unused-but-set-variable warning with GCC 8: ../../../src/amd/addrlib/addrinterface.cpp: In function 'int ElemGetExportNorm(ADDR_HANDLE, const ELEM_GETEXPORTNORM_INPUT*)': ../../../src/amd/addrlib/addrinterface.cpp:835:23: warning: variable 'returnCode' set but not used [-Wunused-but-set-variable] ADDR_E_RETURNCODE returnCode = ADDR_OK; ^~~~~~~~~~ Signed-off-by: Kai Wasserbäch Reviewed-by: Marek Olšák --- src/amd/Makefile.addrlib.am | 1 + src/amd/addrlib/addrinterface.cpp | 4 +++- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/src/amd/Makefile.addrlib.am b/src/amd/Makefile.addrlib.am index 75ff7fbcf75..af0daf907d5 100644 --- a/src/amd/Makefile.addrlib.am +++ b/src/amd/Makefile.addrlib.am @@ -24,6 +24,7 @@ ADDRLIB_LIBS = addrlib/libamdgpu_addrlib.la addrlib_libamdgpu_addrlib_la_CPPFLAGS = \ $(DEFINES) \ -I$(top_srcdir)/src/ \ + -I$(top_srcdir)/include \ -I$(srcdir)/common \ -I$(srcdir)/addrlib \ -I$(srcdir)/addrlib/core \ diff --git a/src/amd/addrlib/addrinterface.cpp b/src/amd/addrlib/addrinterface.cpp index 112431e2cb4..9cbeba41d97 100644 --- a/src/amd/addrlib/addrinterface.cpp +++ b/src/amd/addrlib/addrinterface.cpp @@ -36,6 +36,8 @@ #include "addrcommon.h" +#include "util/macros.h" + using namespace Addr; //////////////////////////////////////////////////////////////////////////////////////////////////// @@ -832,7 +834,7 @@ BOOL_32 ADDR_API ElemGetExportNorm( Addr::Lib* pLib = Lib::GetLib(hLib); BOOL_32 enabled = FALSE; - ADDR_E_RETURNCODE returnCode = ADDR_OK; + MAYBE_UNUSED ADDR_E_RETURNCODE returnCode = ADDR_OK; if (pLib != NULL) { -- 2.11.0