Skip to content

Commit

Permalink
Revert r319531 "[SLPVectorizer] Failure to beneficially vectorize 'co…
Browse files Browse the repository at this point in the history
…pyable' elements in integer binary ops."

It causes builds to fail with "Instruction does not dominate all uses" (PR35497).

> Patch tries to improve vectorization of the following code:
>
> void add1(int * __restrict dst, const int * __restrict src) {
>   *dst++ = *src++;
>   *dst++ = *src++ + 1;
>   *dst++ = *src++ + 2;
>   *dst++ = *src++ + 3;
> }
> Allows to vectorize even if the very first operation is not a binary add, but just a load.
>
> Fixed issues related to previous commit.
>
> Reviewers: spatel, mzolotukhin, mkuper, hfinkel, RKSimon, filcab, ABataev
>
> Reviewed By: ABataev, RKSimon
>
> Subscribers: llvm-commits, RKSimon
>
> Differential Revision: https://reviews.llvm.org/D28907

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@319550 91177308-0d34-0410-b5e6-96231b3b80d8
  • Loading branch information
zmodem committed Dec 1, 2017
1 parent 4d8809f commit faed772
Show file tree
Hide file tree
Showing 4 changed files with 275 additions and 490 deletions.
Loading

0 comments on commit faed772

Please sign in to comment.