[llvm-commits] [llvm] r169048 - /llvm/trunk/lib/Transforms/Vectorize/LoopVectorize.cpp

Nadav Rotem nrotem at apple.com
Fri Nov 30 14:37:11 PST 2012


Author: nadav
Date: Fri Nov 30 16:37:11 2012
New Revision: 169048

URL: http://llvm.org/viewvc/llvm-project?rev=169048&view=rev
Log:
minor cleanups

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

Modified: llvm/trunk/lib/Transforms/Vectorize/LoopVectorize.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Vectorize/LoopVectorize.cpp?rev=169048&r1=169047&r2=169048&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Vectorize/LoopVectorize.cpp (original)
+++ llvm/trunk/lib/Transforms/Vectorize/LoopVectorize.cpp Fri Nov 30 16:37:11 2012
@@ -712,7 +712,7 @@
     const SCEV *Sc = SE->getSCEV(Ptr);
 
     if (SE->isLoopInvariant(Sc, OrigLoop)) {
-      DEBUG(dbgs() << "LV1: Adding RT check for a loop invariant ptr:" <<
+      DEBUG(dbgs() << "LV: Adding RT check for a loop invariant ptr:" <<
             *Ptr <<"\n");
       Starts.push_back(Ptr);
       Ends.push_back(Ptr);
@@ -1423,11 +1423,7 @@
 }
 
 bool LoopVectorizationLegality::canVectorize() {
-  if (!TheLoop->getLoopPreheader()) {
-    assert(false && "No preheader!!");
-    DEBUG(dbgs() << "LV: Loop not normalized." << "\n");
-    return false;
-  }
+  assert(TheLoop->getLoopPreheader() && "No preheader!!");
 
   // We can only vectorize single basic block loops.
   unsigned NumBlocks = TheLoop->getNumBlocks();
@@ -2008,9 +2004,8 @@
     case Instruction::AShr:
     case Instruction::And:
     case Instruction::Or:
-    case Instruction::Xor: {
+    case Instruction::Xor:
       return VTTI->getArithmeticInstrCost(I->getOpcode(), VectorTy);
-    }
     case Instruction::Select: {
       SelectInst *SI = cast<SelectInst>(I);
       const SCEV *CondSCEV = SE->getSCEV(SI->getCondition());





More information about the llvm-commits mailing list