[llvm] [llvm] Issues found PVS studio static analyzer. LoopUnrollAndJam.cpp (PR #170162)

via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 1 07:57:00 PST 2025


https://github.com/Bhuvan1527 created https://github.com/llvm/llvm-project/pull/170162

Minor misoptimization. The PVS-Studio warning: V791 The initial value of the index in the nested loop equals 'I'. Perhaps, 'I + 1' should be used instead. LoopUnrollAndJam.cpp 793

>From 54a8cae58c141443666058a240ab06c0906bbad3 Mon Sep 17 00:00:00 2001
From: bhuvan1527 <balabhuvanvarma at gmail.com>
Date: Mon, 1 Dec 2025 21:22:41 +0530
Subject: [PATCH] [llvm] Issues found PVS studio static analyzer.
 LoopUnrollAndJam.cpp

Minor misoptimization. The PVS-Studio warning: V791 The initial value of the index in the nested loop equals 'I'. Perhaps, 'I + 1' should be used instead. LoopUnrollAndJam.cpp 793
---
 llvm/lib/Transforms/Utils/LoopUnrollAndJam.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/llvm/lib/Transforms/Utils/LoopUnrollAndJam.cpp b/llvm/lib/Transforms/Utils/LoopUnrollAndJam.cpp
index 1e614bd29ee6e..663e45e89c5da 100644
--- a/llvm/lib/Transforms/Utils/LoopUnrollAndJam.cpp
+++ b/llvm/lib/Transforms/Utils/LoopUnrollAndJam.cpp
@@ -789,7 +789,7 @@ checkDependencies(Loop &Root, const BasicBlockSet &SubLoopBlocks,
 
     size_t NumInsts = CurrentLoadsAndStores.size();
     for (size_t I = 0; I < NumInsts; ++I) {
-      for (size_t J = I; J < NumInsts; ++J) {
+      for (size_t J = I + 1; J < NumInsts; ++J) {
         if (!checkDependency(CurrentLoadsAndStores[I], CurrentLoadsAndStores[J],
                              LoopDepth, CurLoopDepth, true, DI))
           return false;



More information about the llvm-commits mailing list