[clang-tools-extra] 9f82209 - [clang-tidy][NFC] Fix clang-analyzer-core.UndefinedBinaryOperatorResult findings

Piotr Zegar via cfe-commits cfe-commits at lists.llvm.org
Sun Aug 27 05:00:06 PDT 2023


Author: Piotr Zegar
Date: 2023-08-27T11:59:03Z
New Revision: 9f8220969cc8c52565d6f9b932e1934513e0c1bb

URL: https://github.com/llvm/llvm-project/commit/9f8220969cc8c52565d6f9b932e1934513e0c1bb
DIFF: https://github.com/llvm/llvm-project/commit/9f8220969cc8c52565d6f9b932e1934513e0c1bb.diff

LOG: [clang-tidy][NFC] Fix clang-analyzer-core.UndefinedBinaryOperatorResult findings

Fix issues found by clang-tidy in clang-tidy source directory.

Added: 
    

Modified: 
    clang-tools-extra/clang-tidy/altera/UnrollLoopsCheck.cpp

Removed: 
    


################################################################################
diff  --git a/clang-tools-extra/clang-tidy/altera/UnrollLoopsCheck.cpp b/clang-tools-extra/clang-tidy/altera/UnrollLoopsCheck.cpp
index e25708d4c05554..197fa812501601 100644
--- a/clang-tools-extra/clang-tidy/altera/UnrollLoopsCheck.cpp
+++ b/clang-tools-extra/clang-tidy/altera/UnrollLoopsCheck.cpp
@@ -171,7 +171,7 @@ bool UnrollLoopsCheck::hasLargeNumIterations(const Stmt *Statement,
   const Stmt *Initializer = ForLoop->getInit();
   const Expr *Conditional = ForLoop->getCond();
   const Expr *Increment = ForLoop->getInc();
-  int InitValue;
+  int InitValue = 0;
   // If the loop variable value isn't known, we can't know the loop bounds.
   if (const auto *InitDeclStatement = dyn_cast<DeclStmt>(Initializer)) {
     if (const auto *VariableDecl =
@@ -183,12 +183,12 @@ bool UnrollLoopsCheck::hasLargeNumIterations(const Stmt *Statement,
     }
   }
 
-  int EndValue;
+  int EndValue = 0;
   const auto *BinaryOp = cast<BinaryOperator>(Conditional);
   if (!extractValue(EndValue, BinaryOp, Context))
     return true;
 
-  double Iterations;
+  double Iterations = 0.0;
 
   // If increment is unary and not one of ++, --, we can't know the loop bounds.
   if (const auto *Op = dyn_cast<UnaryOperator>(Increment)) {


        


More information about the cfe-commits mailing list