Do clang and icc also support __attribute__ ((unused))? If not support, maybe could use IF_DEBUG macro to define the debug only variables.
> -----Original Message----- > From: Beignet [mailto:beignet-boun...@lists.freedesktop.org] On Behalf Of > xionghu....@intel.com > Sent: Thursday, May 7, 2015 09:47 > To: beignet@lists.freedesktop.org > Cc: Luo, Xionghu > Subject: [Beignet] [PATCH 1/2] fix newer gcc warning "variable set but not > used". > > From: Luo Xionghu <xionghu....@intel.com> > > this warning appeared in gcc 4.9.1, declear the variables with attribute > ((unused)). > > Signed-off-by: Luo Xionghu <xionghu....@intel.com> > --- > backend/src/backend/program.cpp | 3 ++- > backend/src/llvm/ExpandLargeIntegers.cpp | 3 ++- > backend/src/llvm/llvm_bitcode_link.cpp | 3 ++- > backend/src/llvm/llvm_printf_parser.cpp | 1 - > 4 files changed, 6 insertions(+), 4 deletions(-) > > diff --git a/backend/src/backend/program.cpp > b/backend/src/backend/program.cpp index eee7c3c..9e0bc66 100644 > --- a/backend/src/backend/program.cpp > +++ b/backend/src/backend/program.cpp > @@ -655,7 +655,8 @@ namespace gbe { > std::string hdirs = OCL_HEADER_FILE_DIR; > std::istringstream hidirs(hdirs); > std::string headerFilePath; > - bool findOcl = false; > + bool findOcl __attribute__ ((unused)); > + findOcl = false; > > while (getline(hidirs, headerFilePath, ':')) { > std::string oclDotHName = headerFilePath + "/ocl.h"; diff --git > a/backend/src/llvm/ExpandLargeIntegers.cpp > b/backend/src/llvm/ExpandLargeIntegers.cpp > index f7e59a5..d7440aa 100644 > --- a/backend/src/llvm/ExpandLargeIntegers.cpp > +++ b/backend/src/llvm/ExpandLargeIntegers.cpp > @@ -442,7 +442,8 @@ static void convertInstruction(Instruction *Inst, > ConversionState &State, > } else if (TruncInst *Trunc = dyn_cast<TruncInst>(Inst)) { > Value *Operand = Trunc->getOperand(0); > assert(shouldConvert(Operand) && "TruncInst is expandable but not its > op"); > - TypePair OpTys = getExpandedIntTypes(Operand->getType()); > + TypePair OpTys __attribute__ ((unused)); > + OpTys = getExpandedIntTypes(Operand->getType()); > ValuePair Ops = State.getConverted(Operand); > if (!shouldConvert(Inst)) { > Value *NewInst = IRB.CreateTrunc(Ops.Lo, Trunc->getType(), Name); diff > --git a/backend/src/llvm/llvm_bitcode_link.cpp > b/backend/src/llvm/llvm_bitcode_link.cpp > index ebf4386..c9824e7 100644 > --- a/backend/src/llvm/llvm_bitcode_link.cpp > +++ b/backend/src/llvm/llvm_bitcode_link.cpp > @@ -51,7 +51,8 @@ namespace gbe > std::string bitCodeFiles = OCL_BITCODE_LIB_PATH; > std::istringstream bitCodeFilePath(bitCodeFiles); > std::string FilePath; > - bool findBC = false; > + bool findBC __attribute__ ((unused)); > + findBC = false; > Module* oclLib = NULL; > SMDiagnostic Err; > > diff --git a/backend/src/llvm/llvm_printf_parser.cpp > b/backend/src/llvm/llvm_printf_parser.cpp > index 2f85443..712c478 100644 > --- a/backend/src/llvm/llvm_printf_parser.cpp > +++ b/backend/src/llvm/llvm_printf_parser.cpp > @@ -503,7 +503,6 @@ error: > info.printf_fmt = printf_fmt; > > sizeof_size = 0; > - int i = 1; > for (auto &s : (*printf_fmt).first) { > int sz = 0; > if (s.type == PRINTF_SLOT_TYPE_STRING) > -- > 2.1.0 > > _______________________________________________ > Beignet mailing list > Beignet@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/beignet _______________________________________________ Beignet mailing list Beignet@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/beignet
