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

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen 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,
----------------
cameron.mcinally wrote:
> Nit: Yank spelling of `scalarize`?
You're right, I'm mixing up American and British spelling in this patch, thanks for spotting!


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

https://reviews.llvm.org/D91077