[llvm] 1b4a173 - [LV] Remove unneeded LoopScalarBody member variable. (NFC)
Florian Hahn via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 16 20:17:44 PDT 2024
Author: Florian Hahn
Date: 2024-10-17T04:15:47+01:00
New Revision: 1b4a173fa41e02eddec9f1cf41324aa4ea8a7fa5
URL: https://github.com/llvm/llvm-project/commit/1b4a173fa41e02eddec9f1cf41324aa4ea8a7fa5
DIFF: https://github.com/llvm/llvm-project/commit/1b4a173fa41e02eddec9f1cf41324aa4ea8a7fa5.diff
LOG: [LV] Remove unneeded LoopScalarBody member variable. (NFC)
Added:
Modified:
llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
Removed:
################################################################################
diff --git a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
index 8bf92f3480620a..3e8bc1451f6243 100644
--- a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
+++ b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
@@ -638,9 +638,6 @@ class InnerLoopVectorizer {
/// there can be multiple exiting edges reaching this block.
BasicBlock *LoopExitBlock;
- /// The scalar loop body.
- BasicBlock *LoopScalarBody;
-
/// A list of all bypass blocks. The first block is the entry of the loop.
SmallVector<BasicBlock *, 4> LoopBypassBlocks;
@@ -2530,7 +2527,6 @@ BasicBlock *InnerLoopVectorizer::emitMemRuntimeChecks(BasicBlock *Bypass) {
}
void InnerLoopVectorizer::createVectorLoopSkeleton(StringRef Prefix) {
- LoopScalarBody = OrigLoop->getHeader();
LoopVectorPreHeader = OrigLoop->getLoopPreheader();
assert(LoopVectorPreHeader && "Invalid loop structure");
LoopExitBlock = OrigLoop->getUniqueExitBlock(); // may be nullptr
@@ -2944,7 +2940,7 @@ void InnerLoopVectorizer::fixVectorizedLoop(VPTransformState &State,
// Set/update profile weights for the vector and remainder loops as original
// loop iterations are now distributed among them. Note that original loop
- // represented by LoopScalarBody becomes remainder loop after vectorization.
+ // becomes the scalar remainder loop after vectorization.
//
// For cases like foldTailByMasking() and requiresScalarEpiloque() we may
// end up getting slightly roughened result but that should be OK since
@@ -2952,12 +2948,11 @@ void InnerLoopVectorizer::fixVectorizedLoop(VPTransformState &State,
// vector code caused by legality checks is ignored, assigning all the weight
// to the vector loop, optimistically.
//
- // For scalable vectorization we can't know at compile time how many iterations
- // of the loop are handled in one vector iteration, so instead assume a pessimistic
- // vscale of '1'.
- Loop *ScalarLoop = LI->getLoopFor(LoopScalarBody);
+ // For scalable vectorization we can't know at compile time how many
+ // iterations of the loop are handled in one vector iteration, so instead
+ // assume a pessimistic vscale of '1'.
Loop *VectorLoop = LI->getLoopFor(HeaderBB);
- setProfileInfoAfterUnrolling(ScalarLoop, VectorLoop, ScalarLoop,
+ setProfileInfoAfterUnrolling(OrigLoop, VectorLoop, OrigLoop,
VF.getKnownMinValue() * UF);
}
More information about the llvm-commits
mailing list