[PATCH] D110640: [SLPVectorizer] Fix crash in isShuffle with scalable vectors

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added a comment.

In D110640#3030337 <https://reviews.llvm.org/D110640#3030337>, @sdesmalen wrote:

> In D110640#3029995 <https://reviews.llvm.org/D110640#3029995>, @ABataev wrote:
>
>> Just check for scalable vectors before this `isShuffle` function call.
>
> Alternatively, do we want to rename `isShuffle` to `isFixedVectorShuffle` and return `None` if the extract-element's vector operand type is scalable?

Yep, that's a good option too.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110640