[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