[PATCH] D148773: [SLP] Make computeExtractCost() more conservative by checking for non-extract values

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 20 03:48:29 PDT 2023


ABataev added a comment.

This is not correct. There are 2 reasons for the vectorization in this: 1. InsertElement cost to position 0 is 0, though it should be non-zero. 2. The cost of the node with the same values as another gather node must be 0, currently it is -1. I have a patch, will send it for review in couple hours.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148773



More information about the llvm-commits mailing list