diff options
author | Simon Pilgrim <llvm-dev@redking.me.uk> | 2018-01-30 18:10:21 +0000 |
---|---|---|
committer | Simon Pilgrim <llvm-dev@redking.me.uk> | 2018-01-30 18:10:21 +0000 |
commit | 073f089c6e5ba77bc398cc2d1b106cebbbd2429f (patch) | |
tree | 8c8314af83dc17d0746cf202831b35f43c314191 /llvm/lib | |
parent | a9f488588dea9b64727b4bbea078ad63fa5829a8 (diff) | |
download | bcm5719-llvm-073f089c6e5ba77bc398cc2d1b106cebbbd2429f.tar.gz bcm5719-llvm-073f089c6e5ba77bc398cc2d1b106cebbbd2429f.zip |
[X86][XOP] Update isVectorShiftByScalarCheap with cases covered by XOP
Similar to D42437, XOP supports variable shift for v16i8/v8i16/v4i32/v2i64 types.
Differential Revision: https://reviews.llvm.org/D42526
llvm-svn: 323797
Diffstat (limited to 'llvm/lib')
-rw-r--r-- | llvm/lib/Target/X86/X86ISelLowering.cpp | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp index 925796cae7a..4c80b86d449 100644 --- a/llvm/lib/Target/X86/X86ISelLowering.cpp +++ b/llvm/lib/Target/X86/X86ISelLowering.cpp @@ -25593,6 +25593,11 @@ bool X86TargetLowering::isVectorShiftByScalarCheap(Type *Ty) const { if (Bits == 8) return false; + // XOP has v16i8/v8i16/v4i32/v2i64 variable vector shifts. + if (Subtarget.hasXOP() && Ty->getPrimitiveSizeInBits() == 128 && + (Bits == 8 || Bits == 16 || Bits == 32 || Bits == 64)) + return false; + // AVX2 has vpsllv[dq] instructions (and other shifts) that make variable // shifts just as cheap as scalar ones. if (Subtarget.hasAVX2() && (Bits == 32 || Bits == 64)) |