[PATCH] D119536: [SLP] Extract intermediate insertelement for external use

Anton Afanasyev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 12 01:03:01 PST 2022


anton-afanasyev added a comment.

In D119536#3314640 <https://reviews.llvm.org/D119536#3314640>, @ABataev wrote:

> Hmm, looks like the problem appears only if we have insertelements to be vectorized not from `vectorizeInsertElementInst`, where we have a proper check. Maybe just add a check for insertelements in other functions and just expect them from `vectorizeInsertElementInst`?

Well, that would definitely solve the problem. There is also another way: on the contrary, remove `hasOneUse()` check from `vectorizeInsertElementInst`. This patch allows it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119536



More information about the llvm-commits mailing list