https://llvm.org/bugs/show_bug.cgi?id=26734

Matthew Simpson <mssim...@codeaurora.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #3 from Matthew Simpson <mssim...@codeaurora.org> ---
This should be fixed as of r262624 (test case included with the fix).

We should not have tried to vectorize this code because the incoming phi value
corresponding to the loop latch block (the previous value) is not defined
inside the loop. It's defined in the preheader. This led to the dominance
problem Michael discovered.

The fix ensures the previous value is defined inside the loop when checking for
first-order recurrences.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs

Reply via email to