diff options
author | Alexey Bataev <a.bataev@hotmail.com> | 2017-02-23 13:37:09 +0000 |
---|---|---|
committer | Alexey Bataev <a.bataev@hotmail.com> | 2017-02-23 13:37:09 +0000 |
commit | f77d1656afb8599ac36b0b192b5fa5c9c721bd93 (patch) | |
tree | 114e640eaef0e9a36b604641f0a578090693a5fe /llvm/lib/Target | |
parent | a606713c330f27329991361fc485ed9908f29983 (diff) | |
download | bcm5719-llvm-f77d1656afb8599ac36b0b192b5fa5c9c721bd93.tar.gz bcm5719-llvm-f77d1656afb8599ac36b0b192b5fa5c9c721bd93.zip |
[SLP] Fix for PR32036: Vectorized horizontal reduction returning wrong
result
Summary:
If the same value is used several times as an extra value, SLP
vectorizer takes it into account only once instead of actual number of
using.
For example:
```
int val = 1;
for (int y = 0; y < 8; y++) {
for (int x = 0; x < 8; x++) {
val = val + input[y * 8 + x] + 3;
}
}
```
We have 2 extra rguments: `1` - initial value of horizontal reduction
and `3`, which is added 8*8 times to the reduction. Before the patch we
added `1` to the reduction value and added once `3`, though it must be
added 64 times.
Reviewers: mkuper, mzolotukhin
Subscribers: llvm-commits
Differential Revision: https://reviews.llvm.org/D30262
llvm-svn: 295972
Diffstat (limited to 'llvm/lib/Target')
0 files changed, 0 insertions, 0 deletions