[SLPVectorizer] Add initial alternate opcode support for cast instructions. (REAPPLIED)
authorSimon Pilgrim <llvm-dev@redking.me.uk>
Wed, 11 Jul 2018 15:05:10 +0000 (15:05 +0000)
committerSimon Pilgrim <llvm-dev@redking.me.uk>
Wed, 11 Jul 2018 15:05:10 +0000 (15:05 +0000)
commit876e99bf2cf8ed3ae729d42755d3658dc5ab2b3f
tree72c58a7bdd9c075a1823e5b9a2507911d5d1b01c
parent495adf52db11fe3d73713734eb6e3fbfa7de7130
[SLPVectorizer] Add initial alternate opcode support for cast instructions. (REAPPLIED)

We currently only support binary instructions in the alternate opcode shuffles.

This patch is an initial attempt at adding cast instructions as well, this raises several issues that we probably want to address as we continue to generalize the alternate mechanism:

1 - Duplication of cost determination - we should probably add scalar/vector costs helper functions and get BoUpSLP::getEntryCost to use them instead of determining costs directly.
2 - Support alternate instructions with the same opcode (e.g. casts with different src types) - alternate vectorization of calls with different IntrinsicIDs will require this.
3 - Allow alternates to be a different instruction type - mixing binary/cast/call etc.
4 - Allow passthrough of unsupported alternate instructions - related to PR30787/D28907 'copyable' elements.

Reapplied with fix to only accept 2 different casts if they come from the same source type.

Differential Revision: https://reviews.llvm.org/D49135

llvm-svn: 336812
llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
llvm/test/Transforms/SLPVectorizer/X86/alternate-cast.ll