[PATCH] D133891: [SLP]Improve isUndefVector function by adding insertelement analysis.

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 20 14:18:43 PDT 2022


ABataev added a comment.

In D133891#3803891 <https://reviews.llvm.org/D133891#3803891>, @srj wrote:

> The line numbers in the ASAN trace are slightly different from this patch, but the culprit seems to be:
>
>   SmallVector<int> InsertMask(NumElts, UndefMaskElem);
>   copy(Mask, std::next(InsertMask.begin(), OffsetBeg));    <-- THIS LINE
>
> Not sure about the semantics here, but presumably InsertMask doesn't have the capacity for the copy and gets tromped on...

Ok, thanks, will add an assertion and check what's wrong here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133891



More information about the llvm-commits mailing list