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

[Alexey Bataev via Phabricator via llvm-commits]

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