[PATCH] D91060: [LoopVectorizer] NFC: Remove unnecessary asserts that VF cannot be scalable.

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

As far as I can tell these do sound OK, except perhaps for the one I put a question on.



================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:4896
                                                          ElementCount VF) {
-  assert(!VF.isScalable() && "scalable vectors not yet supported.");
   if (!blockNeedsPredication(I->getParent()))
----------------
What happens if this function returns true? Do (predicated) scalars work OK for scalable vectors.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91060