[PATCH] D28907: [SLP] Fix for PR30787: Failure to beneficially vectorize 'copyable' elements in integer binary ops.
Dinar Temirbulatov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Dec 16 07:07:55 PST 2017
dtemirbulatov added inline comments.
================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:3244
isJumbled = true;
- LI = cast<LoadInst>(E->Scalars[0]);
- } else {
- LI = cast<LoadInst>(VL0);
- }
+ LI = cast<LoadInst>(E->Scalars[0]);
----------------
We used E->VL0 here before, which is not correct because it could point to a wrong element.
https://reviews.llvm.org/D28907
More information about the llvm-commits
mailing list