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

[Duncan Sands]

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()


> +            (T2->isPointerTy() ||
> +              (T2->isVectorTy()&&  T2->getScalarType()->isPointerTy()))

Similarly.

Ciao, Duncan.