[PATCH] D83779: [SLP] Fix order of `insertelement`/`insertvalue` seed operands
Anton Afanasyev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 5 06:51:31 PDT 2020
anton-afanasyev added a comment.
Fixed
================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:7090
+ auto *IE = dyn_cast<InsertElementInst>(InsertInst);
+ if (IE)
+ return IE->getType()->getNumElements();
----------------
RKSimon wrote:
> if (auto *IE = dyn_cast<InsertElementInst>(InsertInst))
Sure, fixed.
================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:7094
+ auto *IV = cast<InsertValueInst>(InsertInst);
+ AggregateSize = 1;
+ Type *CurrentType = IV->getType();
----------------
RKSimon wrote:
> unsigned AggregateSize = 1;
Fixed.
================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:7121
+ auto *IE = dyn_cast<InsertElementInst>(InsertInst);
+ if (IE) {
+ if (auto *CI = dyn_cast<ConstantInt>(IE->getOperand(2))) {
----------------
RKSimon wrote:
> if (auto *IE = dyn_cast<InsertElementInst>(InsertInst))
Fixed.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83779/new/
https://reviews.llvm.org/D83779
More information about the llvm-commits
mailing list