[llvm-commits] [llvm] r154734 - in /llvm/trunk: include/llvm/Transforms/Vectorize.h lib/Transforms/Vectorize/BBVectorize.cpp test/Transforms/BBVectorize/simple-ldstr-ptrs.ll

[Hal Finkel]

----- Original Message -----
> From: "Duncan Sands" <baldrick at free.fr>
> To: llvm-commits at cs.uiuc.edu
> Sent: Saturday, April 14, 2012 12:46:09 PM
> Subject: Re: [llvm-commits] [llvm] r154734 - in /llvm/trunk: include/llvm/Transforms/Vectorize.h
> lib/Transforms/Vectorize/BBVectorize.cpp test/Transforms/BBVectorize/simple-ldstr-ptrs.ll
> Hi Hal,
> 
> > --- llvm/trunk/lib/Transforms/Vectorize/BBVectorize.cpp (original)
> > +++ llvm/trunk/lib/Transforms/Vectorize/BBVectorize.cpp Sat Apr 14
> > 02:32:43 2012
> > @@ -597,6 +612,14 @@
> >           && (T1->isFPOrFPVectorTy() || T2->isFPOrFPVectorTy()))
> >         return false;
> >
> > + if ((!Config.VectorizePointers || TD == 0)
> 
> > +&& ((T1->isPointerTy() ||
> > + (T1->isVectorTy()&& T1->getScalarType()->isPointerTy())) ||
> 
> ^ I think these two lines can just be:
> T1->getScalarType()->isPointerTy()

I believe that you're correct, thanks! I'll fix this tomorrow.

 -Hal

> 
> 
> > + (T2->isPointerTy() ||
> > + (T2->isVectorTy()&& T2->getScalarType()->isPointerTy()))
> 
> Similarly.
> 
> Ciao, Duncan.
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits