[PATCH] D83779: [SLP] Fix order of `insertelement`/`insertvalue` seed operands

[Anton Afanasyev via Phabricator via llvm-commits]

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