[PATCH] D96791: [SLP] Double UserCost compensation for vector store of aggregate

Anton Afanasyev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 18 01:18:26 PST 2021


anton-afanasyev added a comment.

In D96791#2565982 <https://reviews.llvm.org/D96791#2565982>, @ABataev wrote:

> I think the better approach would be to pass the list of `InsertUses` as a `buildTree` function `UserIgnoreLst` argument. You just need to correctly generate ExtractElement instructions for these InsertElements because currently the compiler just crashes trying to remove instructions used as operands for the original InsertElements. Thoughts?

Thanks, I'm to try this approach.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96791



More information about the llvm-commits mailing list