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

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 29 05:51:56 PDT 2021


ABataev added a comment.

In D110640#3029994 <https://reviews.llvm.org/D110640#3029994>, @kmclaughlin wrote:

>> In D110640#3028021 <https://reviews.llvm.org/D110640#3028021>, @ABataev wrote: 
>> I think the current implementation is still good. Also, isShuffle is called in several places, which one causes the crash?
>
> It's the isShuffle call in `isFullyVectorizableTinyTree` that caused the crash here, which was reached through `vectorizeGEPIndices`->`tryToVectorizeList`.

Just check for scalable vectors before this `isShuffle` function call.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110640



More information about the llvm-commits mailing list