[PATCH] D93063: [LV] Disable epilogue vectorization for scalable VFs

[Cullen Rhodes via Phabricator via llvm-commits]

c-rhodes marked an inline comment as done.
c-rhodes added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:5778
+  // vectors when the loop has a hint to enable vectorization for a given VF.
+  assert(!MainLoopVF.isScalable() && "scalable vectors not yet supported");
+
----------------
bmahjour wrote:
> We can just return `Result` around line 5792 (please see suggestion below), then we don't need to change the check near line 9209.
> We can just return `Result` around line 5792 (please see suggestion below), then we don't need to change the check near line 9209.

That makes more sense, cheers. Added a simple test as well.


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

https://reviews.llvm.org/D93063