From 48d5e750a8189c55087333d2bbc5dd0e1e07ddfa Mon Sep 17 00:00:00 2001 From: Duncan Sands Date: Mon, 10 Jun 2013 12:09:30 +0000 Subject: [PATCH] Avoid warnings about unused parameters that tend to come up a lot when building outside projects with a different compiler than that used to build LLVM itself (eg switching between gcc and clang). git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@183650 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/llvm/ADT/FoldingSet.h | 5 +++-- .../llvm/Support/GetElementPtrTypeIterator.h | 2 +- include/llvm/Support/MathExtras.h | 4 ++-- include/llvm/Support/TargetRegistry.h | 21 +++++++++--------- include/llvm/Target/TargetLowering.h | 22 +++++++++---------- include/llvm/Target/TargetMachine.h | 4 ++-- 6 files changed, 30 insertions(+), 28 deletions(-) diff --git a/include/llvm/ADT/FoldingSet.h b/include/llvm/ADT/FoldingSet.h index 91794dea698..1b2c94c35f8 100644 --- a/include/llvm/ADT/FoldingSet.h +++ b/include/llvm/ADT/FoldingSet.h @@ -352,7 +352,8 @@ template class FoldingSetBucketIterator; template inline bool DefaultFoldingSetTrait::Equals(T &X, const FoldingSetNodeID &ID, - unsigned IDHash, FoldingSetNodeID &TempID) { + unsigned /*IDHash*/, + FoldingSetNodeID &TempID) { FoldingSetTrait::Profile(X, TempID); return TempID == ID; } @@ -366,7 +367,7 @@ template inline bool DefaultContextualFoldingSetTrait::Equals(T &X, const FoldingSetNodeID &ID, - unsigned IDHash, + unsigned /*IDHash*/, FoldingSetNodeID &TempID, Ctx Context) { ContextualFoldingSetTrait::Profile(X, TempID, Context); diff --git a/include/llvm/Support/GetElementPtrTypeIterator.h b/include/llvm/Support/GetElementPtrTypeIterator.h index 5a90553a007..aacb53162cf 100644 --- a/include/llvm/Support/GetElementPtrTypeIterator.h +++ b/include/llvm/Support/GetElementPtrTypeIterator.h @@ -105,7 +105,7 @@ namespace llvm { template inline generic_gep_type_iterator - gep_type_end(Type *Op0, ArrayRef A) { + gep_type_end(Type * /*Op0*/, ArrayRef A) { return generic_gep_type_iterator::end(A.end()); } } // end namespace llvm diff --git a/include/llvm/Support/MathExtras.h b/include/llvm/Support/MathExtras.h index be76f321557..a65bfbcd203 100644 --- a/include/llvm/Support/MathExtras.h +++ b/include/llvm/Support/MathExtras.h @@ -45,7 +45,7 @@ enum ZeroBehavior { template typename enable_if_c::is_integer && !std::numeric_limits::is_signed, std::size_t>::type -countTrailingZeros(T Val, ZeroBehavior ZB = ZB_Width) { +countTrailingZeros(T Val, ZeroBehavior /*ZB*/ = ZB_Width) { if (!Val) return std::numeric_limits::digits; if (Val & 0x1) @@ -114,7 +114,7 @@ inline std::size_t countTrailingZeros(uint64_t Val, ZeroBehavior ZB) { template typename enable_if_c::is_integer && !std::numeric_limits::is_signed, std::size_t>::type -countLeadingZeros(T Val, ZeroBehavior ZB = ZB_Width) { +countLeadingZeros(T Val, ZeroBehavior /*ZB*/ = ZB_Width) { if (!Val) return std::numeric_limits::digits; diff --git a/include/llvm/Support/TargetRegistry.h b/include/llvm/Support/TargetRegistry.h index c9d43738fac..109b3a2fa3b 100644 --- a/include/llvm/Support/TargetRegistry.h +++ b/include/llvm/Support/TargetRegistry.h @@ -891,7 +891,7 @@ namespace llvm { TargetRegistry::RegisterMCAsmInfo(T, &Allocator); } private: - static MCAsmInfo *Allocator(const MCRegisterInfo &MRI, StringRef TT) { + static MCAsmInfo *Allocator(const MCRegisterInfo &/*MRI*/, StringRef TT) { return new MCAsmInfoImpl(TT); } @@ -925,8 +925,9 @@ namespace llvm { TargetRegistry::RegisterMCCodeGenInfo(T, &Allocator); } private: - static MCCodeGenInfo *Allocator(StringRef TT, Reloc::Model RM, - CodeModel::Model CM, CodeGenOpt::Level OL) { + static MCCodeGenInfo *Allocator(StringRef /*TT*/, Reloc::Model /*RM*/, + CodeModel::Model /*CM*/, + CodeGenOpt::Level /*OL*/) { return new MCCodeGenInfoImpl(); } }; @@ -1025,7 +1026,7 @@ namespace llvm { TargetRegistry::RegisterMCRegInfo(T, &Allocator); } private: - static MCRegisterInfo *Allocator(StringRef TT) { + static MCRegisterInfo *Allocator(StringRef /*TT*/) { return new MCRegisterInfoImpl(); } }; @@ -1058,8 +1059,8 @@ namespace llvm { TargetRegistry::RegisterMCSubtargetInfo(T, &Allocator); } private: - static MCSubtargetInfo *Allocator(StringRef TT, StringRef CPU, - StringRef FS) { + static MCSubtargetInfo *Allocator(StringRef /*TT*/, StringRef /*CPU*/, + StringRef /*FS*/) { return new MCSubtargetInfoImpl(); } }; @@ -1178,10 +1179,10 @@ namespace llvm { } private: - static MCCodeEmitter *Allocator(const MCInstrInfo &II, - const MCRegisterInfo &MRI, - const MCSubtargetInfo &STI, - MCContext &Ctx) { + static MCCodeEmitter *Allocator(const MCInstrInfo &/*II*/, + const MCRegisterInfo &/*MRI*/, + const MCSubtargetInfo &/*STI*/, + MCContext &/*Ctx*/) { return new MCCodeEmitterImpl(); } }; diff --git a/include/llvm/Target/TargetLowering.h b/include/llvm/Target/TargetLowering.h index d67e55dc665..9483282e0b7 100644 --- a/include/llvm/Target/TargetLowering.h +++ b/include/llvm/Target/TargetLowering.h @@ -148,7 +148,7 @@ public: // Return the pointer type for the given address space, defaults to // the pointer type from the data layout. // FIXME: The default needs to be removed once all the code is updated. - virtual MVT getPointerTy(uint32_t AS = 0) const { return PointerTy; } + virtual MVT getPointerTy(uint32_t /*AS*/ = 0) const { return PointerTy; } virtual MVT getScalarShiftAmountTy(EVT LHSTy) const; EVT getShiftAmountTy(EVT LHSTy) const; @@ -157,12 +157,14 @@ public: /// this target. bool isSelectExpensive() const { return SelectIsExpensive; } - virtual bool isSelectSupported(SelectSupportKind kind) const { return true; } + virtual bool isSelectSupported(SelectSupportKind /*kind*/) const { + return true; + } /// shouldSplitVectorElementType - Return true if a vector of the given type /// should be split (TypeSplitVector) instead of promoted /// (TypePromoteInteger) during type legalization. - virtual bool shouldSplitVectorElementType(EVT VT) const { return false; } + virtual bool shouldSplitVectorElementType(EVT /*VT*/) const { return false; } /// isIntDivCheap() - Return true if integer divide is usually cheaper than /// a sequence of several shifts, adds, and multiplies for this target. @@ -694,7 +696,7 @@ public: /// replacements don't generate code that causes an alignment error (trap) on /// the target machine. /// @brief Determine if the target supports unaligned memory accesses. - virtual bool allowsUnalignedMemoryAccesses(EVT, bool *Fast = 0) const { + virtual bool allowsUnalignedMemoryAccesses(EVT, bool * /*Fast*/ = 0) const { return false; } @@ -724,9 +726,7 @@ public: /// targets without SSE2 f64 load / store are done with fldl / fstpl which /// also does type conversion. Note the specified type doesn't have to be /// legal as the hook is used before type legalization. - virtual bool isSafeMemOpType(MVT VT) const { - return true; - } + virtual bool isSafeMemOpType(MVT /*VT*/) const { return true; } /// usesUnderscoreSetJmp - Determine if we should use _setjmp or setjmp /// to implement llvm.setjmp. @@ -1110,9 +1110,9 @@ public: /// much computation as possible to be done in the address mode for that /// operand. This hook lets targets also pass back when this should be done /// on intrinsics which load/store. - virtual bool GetAddrModeArguments(IntrinsicInst *I, - SmallVectorImpl &Ops, - Type *&AccessTy) const { + virtual bool GetAddrModeArguments(IntrinsicInst * /*I*/, + SmallVectorImpl &/*Ops*/, + Type *&/*AccessTy*/) const { return false; } @@ -2031,7 +2031,7 @@ public: /// tail call. /// This is used to determine whether it is possible /// to codegen a libcall as tail call at legalization time. - virtual bool isUsedByReturnOnly(SDNode *, SDValue &Chain) const { + virtual bool isUsedByReturnOnly(SDNode *, SDValue &/*Chain*/) const { return false; } diff --git a/include/llvm/Target/TargetMachine.h b/include/llvm/Target/TargetMachine.h index c5e4b278d9f..7f04119ae66 100644 --- a/include/llvm/Target/TargetMachine.h +++ b/include/llvm/Target/TargetMachine.h @@ -252,8 +252,8 @@ public: formatted_raw_ostream &, CodeGenFileType, bool /*DisableVerify*/ = true, - AnalysisID StartAfter = 0, - AnalysisID StopAfter = 0) { + AnalysisID /*StartAfter*/ = 0, + AnalysisID /*StopAfter*/ = 0) { return true; } -- 2.34.1