On Sat, Aug 18, 2018 at 7:16 AM Kai Wasserbäch <k...@dev.carbon-project.org> wrote: > > Only used, when asserts are enabled. > > Fixes an unused-variable warning with GCC 8: > ../../../src/amd/addrlib/r800/egbaddrlib.cpp: In member function 'int > Addr::V1::EgBasedLib::SanityCheckMacroTiled(ADDR_TILEINFO*) const': > ../../../src/amd/addrlib/r800/egbaddrlib.cpp:982:13: warning: unused > variable 'numPipes' [-Wunused-variable] > UINT_32 numPipes = HwlGetPipes(pTileInfo); > ^~~~~~~~ > > Cc: Marek Olšák <mar...@gmail.com> > Signed-off-by: Kai Wasserbäch <k...@dev.carbon-project.org> > --- > src/amd/addrlib/r800/egbaddrlib.cpp | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/src/amd/addrlib/r800/egbaddrlib.cpp > b/src/amd/addrlib/r800/egbaddrlib.cpp > index 3947cfda2f..37a60c7a85 100644 > --- a/src/amd/addrlib/r800/egbaddrlib.cpp > +++ b/src/amd/addrlib/r800/egbaddrlib.cpp > @@ -33,6 +33,8 @@ > > #include "egbaddrlib.h" > > +#include "util/macros.h" > + > namespace Addr > { > namespace V1 > @@ -978,8 +980,8 @@ BOOL_32 EgBasedLib::SanityCheckMacroTiled( > ADDR_TILEINFO* pTileInfo ///< [in] macro-tiled parameters > ) const > { > - BOOL_32 valid = TRUE;
Please don't change the formatting of unrelated code. Marek > - UINT_32 numPipes = HwlGetPipes(pTileInfo); > + BOOL_32 valid = TRUE; > + MAYBE_UNUSED UINT_32 numPipes = HwlGetPipes(pTileInfo); > > switch (pTileInfo->banks) > { > -- > 2.18.0 > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev