[PATCH] Move debug message in vectorizer

Nadav Rotem nrotem at apple.com
Mon Nov 11 08:29:04 PST 2013


LGTM Renato! 

On Nov 11, 2013, at 5:54 AM, Renato Golin <renato.golin at linaro.org> wrote:

> Hi nadav,
> 
> No functional change, just better reporting.
> 
> http://llvm-reviews.chandlerc.com/D2137
> 
> Files:
>  lib/Transforms/Vectorize/LoopVectorize.cpp
> 
> Index: lib/Transforms/Vectorize/LoopVectorize.cpp
> ===================================================================
> --- lib/Transforms/Vectorize/LoopVectorize.cpp
> +++ lib/Transforms/Vectorize/LoopVectorize.cpp
> @@ -965,15 +965,12 @@
>     unsigned UF = CM.selectUnrollFactor(OptForSize, Hints.Unroll, VF.Width,
>                                         VF.Cost);
> 
> -    if (VF.Width == 1) {
> -      DEBUG(dbgs() << "LV: Vectorization is possible but not beneficial.\n");
> -    }
> -
>     DEBUG(dbgs() << "LV: Found a vectorizable loop ("<< VF.Width << ") in "<<
>           F->getParent()->getModuleIdentifier() << '\n');
>     DEBUG(dbgs() << "LV: Unroll Factor is " << UF << '\n');
> 
>     if (VF.Width == 1) {
> +      DEBUG(dbgs() << "LV: Vectorization is possible but not beneficial.\n");
>       if (UF == 1)
>         return false;
>       // We decided not to vectorize, but we may want to unroll.
> <D2137.1.patch>




More information about the llvm-commits mailing list