diff --git a/include/llvm-c/Transforms/Vectorize.h b/include/llvm-c/Transforms/Vectorize.h index 178465ac827..ce92eaac0f5 100644 --- a/include/llvm-c/Transforms/Vectorize.h +++ b/include/llvm-c/Transforms/Vectorize.h @@ -1,4 +1,4 @@ -/*===---------------------------Vectorize.h ------------------- -*- C++ -*-===*\ +/*===---------------------------Vectorize.h --------------------- -*- C -*-===*\ |*===----------- Vectorization Transformation Library C Interface ---------===*| |* *| |* The LLVM Compiler Infrastructure *| diff --git a/lib/Transforms/Vectorize/BBVectorize.cpp b/lib/Transforms/Vectorize/BBVectorize.cpp index 9c2c8dd51b3..6d879b79b0c 100644 --- a/lib/Transforms/Vectorize/BBVectorize.cpp +++ b/lib/Transforms/Vectorize/BBVectorize.cpp @@ -300,6 +300,7 @@ namespace { AU.addRequired(); AU.addPreserved(); AU.addPreserved(); + AU.setPreservesCFG(); } // This returns the vector type that holds a pair of the provided type. @@ -308,9 +309,9 @@ namespace { if (VectorType *VTy = dyn_cast(ElemTy)) { unsigned numElem = VTy->getNumElements(); return VectorType::get(ElemTy->getScalarType(), numElem*2); - } else { - return VectorType::get(ElemTy, 2); } + + return VectorType::get(ElemTy, 2); } // Returns the weight associated with the provided value. A chain of @@ -431,7 +432,7 @@ namespace { std::vector PairableInsts; std::multimap CandidatePairs; getCandidatePairs(BB, CandidatePairs, PairableInsts); - if (PairableInsts.size() == 0) return false; + if (PairableInsts.empty()) return false; // Now we have a map of all of the pairable instructions and we need to // select the best possible pairing. A good pairing is one such that the @@ -444,7 +445,7 @@ namespace { std::multimap ConnectedPairs; computeConnectedPairs(CandidatePairs, PairableInsts, ConnectedPairs); - if (ConnectedPairs.size() == 0) return false; + if (ConnectedPairs.empty()) return false; // Build the pairable-instruction dependency map DenseSet PairableInstUsers; @@ -459,7 +460,7 @@ namespace { choosePairs(CandidatePairs, PairableInsts, ConnectedPairs, PairableInstUsers, ChosenPairs); - if (ChosenPairs.size() == 0) return false; + if (ChosenPairs.empty()) return false; NumFusedOps += ChosenPairs.size(); // A set of pairs has now been selected. It is now necessary to replace the @@ -647,8 +648,8 @@ namespace { UsesI = true; if (!UsesI) - for (User::op_iterator JU = J->op_begin(), e = J->op_end(); - JU != e; ++JU) { + for (User::op_iterator JU = J->op_begin(), JE = J->op_end(); + JU != JE; ++JU) { Value *V = *JU; if (I == V || Users.count(V)) { UsesI = true;