Reviewed-by: Marek Olšák <marek.ol...@amd.com>
Marek On Tue, Aug 21, 2018 at 11:25 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/gfx9/gfx9addrlib.cpp: In member function > 'ADDR_E_RETURNCODE Addr::V2::Gfx9Lib::ComputeStereoInfo(const > ADDR2_COMPUTE_SURFACE_INFO_INPUT*, ADDR2_COMPUTE_SURFACE_INFO_OUTPUT*, > unsigned int*) const': > ../../../src/amd/addrlib/gfx9/gfx9addrlib.cpp:3879:34: warning: unused > variable 'pEqToCheck' [-Wunused-variable] > const ADDR_EQUATION *pEqToCheck = > &m_equationTable[eqIndex]; > ^~~~~~~~~~ > > v2: Don't realign other variable definitions, to keep in line with file > style (Marek) > > Cc: Marek Olšák <mar...@gmail.com> > Signed-off-by: Kai Wasserbäch <k...@dev.carbon-project.org> > Reviewed-by: Marek Olšák <marek.ol...@amd.com> > --- > src/amd/addrlib/gfx9/gfx9addrlib.cpp | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/amd/addrlib/gfx9/gfx9addrlib.cpp > b/src/amd/addrlib/gfx9/gfx9addrlib.cpp > index 853a041158..d27aabbb60 100644 > --- a/src/amd/addrlib/gfx9/gfx9addrlib.cpp > +++ b/src/amd/addrlib/gfx9/gfx9addrlib.cpp > @@ -3878,7 +3878,7 @@ ADDR_E_RETURNCODE Gfx9Lib::ComputeStereoInfo( > const UINT_32 numBankBits = > GetBankXorBits(blkSizeLog2); > const UINT_32 bppLog2 = Log2(pIn->bpp >> 3); > const UINT_32 maxYCoordBlock256 = > Log2(Block256_2d[bppLog2].h) - 1; > - const ADDR_EQUATION *pEqToCheck = > &m_equationTable[eqIndex]; > + MAYBE_UNUSED const ADDR_EQUATION *pEqToCheck = > &m_equationTable[eqIndex]; > > ADDR_ASSERT(maxYCoordBlock256 == > GetMaxValidChannelIndex(&pEqToCheck->addr[0], > GetBlockSizeLog2(ADDR_SW_256B), 1)); > -- > 2.18.0 > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
