[LLVMbugs] [Bug 17662] New: LoopVectorizer: setAlreadyVectorized needs to set width and unroll to one

[bugzilla-daemon at llvm.org]

http://llvm.org/bugs/show_bug.cgi?id=17662

            Bug ID: 17662
           Summary: LoopVectorizer: setAlreadyVectorized needs to set
                    width and unroll to one
           Product: tools
           Version: trunk
          Hardware: PC
                OS: All
            Status: NEW
          Severity: normal
          Priority: P
         Component: opt
          Assignee: unassignedbugs at nondot.org
          Reporter: aschwaighofer at apple.com
                CC: llvmbugs at cs.uiuc.edu
    Classification: Unclassified

/// Mark the loop L as already vectorized by setting the width to 1.
  void setAlreadyVectorized(Loop *L)

We should set width *and* unroll to one. Because this is how we use it:


    LoopVectorizeHints Hints(L, DisableUnrolling);

    if (Hints.Width == 1 && Hints.Unroll == 1) {
      DEBUG(dbgs() << "LV: Not vectorizing.\n");
      return false;
    }

We need to make sure that we set both otherwise we analyze vectorized loops
again.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20131023/0624f875/attachment.html>