[llvm] r308750 - [SLPVectorizer] Replace E->Scalars to VL0 at vectorizeTree and move comment, NFCI.

Dinar Temirbulatov via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 21 09:02:56 PDT 2017


Author: dinar
Date: Fri Jul 21 09:02:56 2017
New Revision: 308750

URL: http://llvm.org/viewvc/llvm-project?rev=308750&view=rev
Log:
[SLPVectorizer] Replace E->Scalars to VL0 at vectorizeTree and move comment, NFCI.

Modified:
    llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp

Modified: llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp?rev=308750&r1=308749&r2=308750&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp (original)
+++ llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp Fri Jul 21 09:02:56 2017
@@ -2746,7 +2746,7 @@ Value *BoUpSLP::vectorizeTree(TreeEntry
     }
     case Instruction::Call: {
       CallInst *CI = cast<CallInst>(VL0);
-      setInsertPointAfterBundle(E->Scalars);
+      setInsertPointAfterBundle(VL0);
       Function *FI;
       Intrinsic::ID IID  = Intrinsic::not_intrinsic;
       Value *ScalarArg = nullptr;
@@ -2986,9 +2986,8 @@ BoUpSLP::vectorizeTree(ExtraValueToDebug
         for (User *U : Scalar->users()) {
           DEBUG(dbgs() << "SLP: \tvalidating user:" << *U << ".\n");
 
-          assert((getTreeEntry(U) ||
-                  // It is legal to replace users in the ignorelist by undef.
-                  is_contained(UserIgnoreList, U)) &&
+          // It is legal to replace users in the ignorelist by undef.
+          assert((getTreeEntry(U) || is_contained(UserIgnoreList, U)) &&
                  "Replacing out-of-tree value with undef");
         }
 #endif




More information about the llvm-commits mailing list