diff --git a/clang/lib/Sema/AnalysisBasedWarnings.cpp b/clang/lib/Sema/AnalysisBasedWarnings.cpp index 61a48546989ca..e1b0d48cfb8d1 100644 --- a/clang/lib/Sema/AnalysisBasedWarnings.cpp +++ b/clang/lib/Sema/AnalysisBasedWarnings.cpp @@ -2558,10 +2558,10 @@ class UnsafeBufferUsageReporter : public UnsafeBufferUsageHandler { } /* TO_UPSTREAM(BoundsSafety) ON */ - void handleUnsafeCountAttributedPointerArgument(const CallExpr *Call, - const Expr *Arg, - bool IsRelatedToDecl, - ASTContext &Ctx) override { + void handleUnsafeCountAttributedPointerArgument( + const CallExpr *Call, const Expr *Arg, + [[maybe_unused]] bool IsRelatedToDecl, + [[maybe_unused]] ASTContext &Ctx) override { const FunctionDecl *FD = Call->getDirectCallee(); const auto *FPT = FD->getType()->getAs(); @@ -2604,14 +2604,16 @@ class UnsafeBufferUsageReporter : public UnsafeBufferUsageHandler { << PtrParamName << CountParamName; } - void handleUnsafeSinglePointerArgument(const Expr *Arg, bool IsRelatedToDecl, - ASTContext &Ctx) override { + void + handleUnsafeSinglePointerArgument(const Expr *Arg, + [[maybe_unused]] bool IsRelatedToDecl, + [[maybe_unused]] ASTContext &Ctx) override { S.Diag(Arg->getBeginLoc(), diag::warn_unsafe_single_pointer_argument); } - void handleTooComplexCountAttributedAssign(const Expr *E, const ValueDecl *VD, - bool IsRelatedToDecl, - ASTContext &Ctx) override { + void handleTooComplexCountAttributedAssign( + const Expr *E, const ValueDecl *VD, [[maybe_unused]] bool IsRelatedToDecl, + [[maybe_unused]] ASTContext &Ctx) override { SourceLocation Loc = E->getBeginLoc(); if (const auto *BO = dyn_cast(E)) Loc = BO->getOperatorLoc(); @@ -2634,11 +2636,10 @@ class UnsafeBufferUsageReporter : public UnsafeBufferUsageHandler { } } - void handleAssignToImmutableObject(const BinaryOperator *Assign, - const ValueDecl *VD, - AssignToImmutableObjectKind Kind, - bool IsRelatedToDecl, - ASTContext &Ctx) override { + void handleAssignToImmutableObject( + const BinaryOperator *Assign, const ValueDecl *VD, + AssignToImmutableObjectKind Kind, [[maybe_unused]] bool IsRelatedToDecl, + [[maybe_unused]] ASTContext &Ctx) override { S.Diag(Assign->getOperatorLoc(), diag::warn_cannot_assign_to_immutable_bounds_attributed_object) << getBoundsAttributedObjectKind(VD) << VD << Kind; @@ -2663,8 +2664,8 @@ class UnsafeBufferUsageReporter : public UnsafeBufferUsageHandler { const Expr *LastAssignInGroup, const llvm::SmallPtrSetImpl &Required, const llvm::SmallPtrSetImpl &Missing, - bool IsRelatedToDecl, ASTContext &Ctx) override { - + [[maybe_unused]] bool IsRelatedToDecl, + [[maybe_unused]] ASTContext &Ctx) override { llvm::SmallString<64> RequiredAssignments = DeclSetToStr(Required); llvm::SmallString<64> MissingAssignments = DeclSetToStr(Missing); auto Loc = @@ -2677,8 +2678,9 @@ class UnsafeBufferUsageReporter : public UnsafeBufferUsageHandler { void handleDuplicatedAssignment(const BinaryOperator *Assign, const BinaryOperator *PrevAssign, - const ValueDecl *VD, bool IsRelatedToDecl, - ASTContext &Ctx) override { + const ValueDecl *VD, + [[maybe_unused]] bool IsRelatedToDecl, + [[maybe_unused]] ASTContext &Ctx) override { S.Diag(Assign->getOperatorLoc(), diag::warn_duplicated_assignment_in_bounds_attributed_group) << getBoundsAttributedObjectKind(VD) << VD;