[PATCH] D101477: [SLP]Fix the crash on cost calculation if non-compatible vectors shuffled.

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 28 11:14:35 PDT 2021


ABataev created this revision.
ABataev added reviewers: spatel, RKSimon, dtemirbulatov, anton-afanasyev.
Herald added subscribers: kerbowa, hiraditya, nhaehnle, jvesely.
ABataev requested review of this revision.
Herald added a project: LLVM.

If the extracts from the non-power-2 vectors are recognized as shuffles,
need some extra checks to not crash cost calculations if trying to gext
the ecost for subvector extracts. In this case need to check carefully
that we do not exit out of bounds of the original vector, otherwise the
TTI's cost model will crash on assert.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D101477

Files:
  llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
  llvm/test/Transforms/SLPVectorizer/AMDGPU/add_sub_sat.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D101477.341272.patch
Type: text/x-patch
Size: 5757 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210428/b6ddd522/attachment.bin>


More information about the llvm-commits mailing list