[PATCH] D31910: [LV] Avoid vectorizing first order recurrences when phi used outside loop

Matthew Simpson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 11 13:28:29 PDT 2017


mssimpso added a comment.

In https://reviews.llvm.org/D31910#724159, @anna wrote:

> @mssimpso: Hi Matt, I've updated the diff to avoid vectorizing the loop when it generates incorrect code gen. Also, there's an AArch64 test case that already showcases this exact problem: generate incorrect code gen in first order recurrence. Please take a look. It's also aptly named `phi_fail2`!


Yeah, that's true. Thanks very much for sorting this out!


https://reviews.llvm.org/D31910





More information about the llvm-commits mailing list