[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