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

Kerry McLaughlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 28 10:30:42 PDT 2021


kmclaughlin added a comment.

Hi @abataev, I've added a fix for this assertion failure in the same place as you suggested on a previous patch, D104809 <https://reviews.llvm.org/D104809>. Since this is quite similar and there would now be two fixes in `buildTree_rec` for the same issue, I was wondering if you think it's worth revisiting the fix @CarolineConcatto first proposed on that patch? Her first revision added the `phiNodeHasScalableOp` & `basicBlockHasScalableOp` functions to avoid vectorizing early if there are any instructions found with scalable types.


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