[PATCH] D26083: [LV] Scalarize operands of predicated instructions
    Gil Rapaport via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Oct 31 14:40:41 PDT 2016
    
    
  
gilr added inline comments.
================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:6546
+    // there's nothing to do.
+    if (ScalarCosts.count(I) || !TheLoop->contains(I))
+      continue;
----------------
It seems the only instructions pushed into Worklist are PredInst and instructions from its basic block. Is the invariance check necessary?
https://reviews.llvm.org/D26083
    
    
More information about the llvm-commits
mailing list