[PATCH] D38697: [SLP] Fix for PR32086: Count InsertElementInstr of the same elements as shuffle.

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 10 08:31:41 PDT 2017


ABataev added a comment.

In https://reviews.llvm.org/D38697#892963, @RKSimon wrote:

> This patch seems to change the blending-shuffle.ll test case in the same way as https://reviews.llvm.org/D38693 - what is the relationship/dependency between them?


Yes, these patches do the same for this particular tests. But https://reviews.llvm.org/D38693 fixes the global problem with the handling of `ExtractElementInst`s, while this one can fix the only pattern used in blending-shuffle.ll. These patches can do the same, but only for limited number of patterns.


https://reviews.llvm.org/D38697





More information about the llvm-commits mailing list