[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