StaticAnalyzer: Fix GCC 6 indention warning in ArrayBoundCheckerV2.cpp
Dear Clang developers,
please find attached a tiny patch which fixes a indentation warning from
GCC 6 within ArrayBoundCheckerV2.cpp
Proposed commit message:
> StaticAnalyzer: Fix GCC 6 indention warning in ArrayBoundCheckerV2.cpp
>
> Patch by Christoph Grüninger
Bye
Christoph
Index: lib/StaticAnalyzer/Checkers/ArrayBoundCheckerV2.cpp
===
--- lib/StaticAnalyzer/Checkers/ArrayBoundCheckerV2.cpp (Revision 278795)
+++ lib/StaticAnalyzer/Checkers/ArrayBoundCheckerV2.cpp (Arbeitskopie)
@@ -157,9 +157,10 @@
// If we are under constrained and the index variables are tainted, report.
if (state_exceedsUpperBound && state_withinUpperBound) {
- if (state->isTainted(rawOffset.getByteOffset()))
+ if (state->isTainted(rawOffset.getByteOffset())) {
reportOOB(checkerContext, state_exceedsUpperBound, OOB_Tainted);
-return;
+ }
+ return;
}
// If we are constrained enough to definitely exceed the upper bound, report.
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[Ping] StaticAnalyzer: Fix GCC 6 indention warning in ArrayBoundCheckerV2.cpp
Hi! I know my patch is of minor importantance. Could still somebody be so kind to review and maybe to apply it to trunk? Thanks Christoph Am 16.08.2016 um 15:41 schrieb Christoph Grüninger via cfe-commits: > Dear Clang developers, > please find attached a tiny patch which fixes a indentation warning from > GCC 6 within ArrayBoundCheckerV2.cpp > > Proposed commit message: >> StaticAnalyzer: Fix GCC 6 indention warning in ArrayBoundCheckerV2.cpp >> >> Patch by Christoph Grüninger > > Bye > Christoph > > > > ___ > cfe-commits mailing list > cfe-commits@lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits > ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[clang] Clang / SemaHLSL: Fix unused variable in release build (PR #124398)
https://github.com/gruenich created
https://github.com/llvm/llvm-project/pull/124398
None
From 29ced5f7bf64c4112879fdf9ed70abd1fc83bad4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Christoph=20Gr=C3=BCninger?=
Date: Sat, 25 Jan 2025 09:10:42 +0100
Subject: [PATCH] Clang / SemaHLSL: Fix unused variable in release build
---
clang/lib/Sema/SemaHLSL.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/clang/lib/Sema/SemaHLSL.cpp b/clang/lib/Sema/SemaHLSL.cpp
index a7033cb54886a7..7ac9acd8314b76 100644
--- a/clang/lib/Sema/SemaHLSL.cpp
+++ b/clang/lib/Sema/SemaHLSL.cpp
@@ -423,7 +423,7 @@ static CXXRecordDecl *createHostLayoutStruct(Sema &S,
LS->startDefinition();
// copy base struct, create HLSL Buffer compatible version if needed
- if (unsigned NumBases = StructDecl->getNumBases()) {
+ if ([[maybe_unused]] unsigned NumBases = StructDecl->getNumBases()) {
assert(NumBases == 1 && "HLSL supports only one base type");
CXXBaseSpecifier Base = *StructDecl->bases_begin();
CXXRecordDecl *BaseDecl = Base.getType()->getAsCXXRecordDecl();
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[clang] Clang / SemaHLSL: Fix unused variable in release build (PR #124398)
gruenich wrote: Already fixed by 7974f12b1e3682514bd58b35c5a784f35938fa04 https://github.com/llvm/llvm-project/pull/124398 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[clang] Clang / SemaHLSL: Fix unused variable in release build (PR #124398)
https://github.com/gruenich closed https://github.com/llvm/llvm-project/pull/124398 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
