[PATCH] D57059: [SLP] Initial support for the vectorization of the non-power-of-2 vectors.

[Valeriy Dmitriev via Phabricator via llvm-commits]

vdmitrie added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:6277
+        Ops = FixedChain;
+      }
 
----------------
1) if -else bodies are exactly the same.
2) With OpsWidth !=VF there is still possibility to bypass it depending on UserCost and AllowReorder values. It should be either assertion to ensure it never happens or "break".


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57059/new/

https://reviews.llvm.org/D57059