[llvm] 8896c36 - [LV] Do not set insert point in completeLoopSkeleton. (NFCI)

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 14 05:21:57 PDT 2022


Author: Florian Hahn
Date: 2022-03-14T12:21:26Z
New Revision: 8896c36624b2eb848b892da2d29eb7a1a41bcc0e

URL: https://github.com/llvm/llvm-project/commit/8896c36624b2eb848b892da2d29eb7a1a41bcc0e
DIFF: https://github.com/llvm/llvm-project/commit/8896c36624b2eb848b892da2d29eb7a1a41bcc0e.diff

LOG: [LV] Do not set insert point in completeLoopSkeleton. (NFCI)

The insertion point for the builder used during VPlan code generation is
set during code generation. Setting the insert point here is dead code
and can be removed.

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 27b8c86730928..4e752f97b43d0 100644
--- a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
+++ b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
@@ -3271,7 +3271,6 @@ BasicBlock *InnerLoopVectorizer::completeLoopSkeleton(Loop *L,
   // Get ready to start creating new instructions into the vectorized body.
   assert(LoopVectorPreHeader == L->getLoopPreheader() &&
          "Inconsistent vector loop preheader");
-  Builder.SetInsertPoint(&*LoopVectorBody->getFirstInsertionPt());
 
 #ifdef EXPENSIVE_CHECKS
   assert(DT->verify(DominatorTree::VerificationLevel::Fast));


        


More information about the llvm-commits mailing list