[PATCH] D86249: [llvm][sve] Make `llvm.masked.[gather|scatter]` legal for SVE.

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

> What is the reason for not being able to scalarize anything that work on scalable vectors? Is it because this scalarization pass cannot (shouldn't?) produce loops?

The scalarization pass currently doesn't produce loops, so the resulting code is nonsense.  We could, in theory, teach the scalarization pass how to generate loops, but it would be hard to do well.

> OK - but I think I should remove them from this patch. We will introduce them when needed. Does that sound right to you?

Makes sense


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86249