[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