[PATCH] D91077: [LoopVectorizer][SVE] Vectorize a simple loop with with a scalable VF.

[Cameron McInally via Phabricator via llvm-commits]

cameron.mcinally added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:8100
   if (State.Instance) { // Generate a single instance.
+    assert(!State.VF.isScalable() && "Can't scalarise a scalable vector");
     State.ILV->scalarizeInstruction(Ingredient, *this, *State.Instance,
----------------
Nit: Yank spelling of `scalarize`?


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:8122
+  assert((!State.VF.isScalable() || IsUniform) &&
+         "Can't scalarise a scalable vector");
   for (unsigned Part = 0; Part < State.UF; ++Part)
----------------
Same here.


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

https://reviews.llvm.org/D91077