diff options
author | Simon Pilgrim <llvm-dev@redking.me.uk> | 2016-06-04 13:42:46 +0000 |
---|---|---|
committer | Simon Pilgrim <llvm-dev@redking.me.uk> | 2016-06-04 13:42:46 +0000 |
commit | fda22d66fc531865fe8c4ebc2fafb39815c9f261 (patch) | |
tree | dce934c6b99c1d1bb2c77ec685a68aed1ac30800 /llvm/lib/Transforms/InstCombine/InstCombineSimplifyDemanded.cpp | |
parent | fe9466fe2ce7ac9b50c09b3f3c2d14370147f4dc (diff) | |
download | bcm5719-llvm-fda22d66fc531865fe8c4ebc2fafb39815c9f261.tar.gz bcm5719-llvm-fda22d66fc531865fe8c4ebc2fafb39815c9f261.zip |
[InstCombine][MMX] Extend SimplifyDemandedUseBits MOVMSK support to MMX
Add the MMX implementation to the SimplifyDemandedUseBits SSE/AVX MOVMSK support added in D19614
Requires a minor tweak as llvm.x86.mmx.pmovmskb takes a x86_mmx argument - so we have to be explicit about the implied v8i8 vector type.
llvm-svn: 271789
Diffstat (limited to 'llvm/lib/Transforms/InstCombine/InstCombineSimplifyDemanded.cpp')
-rw-r--r-- | llvm/lib/Transforms/InstCombine/InstCombineSimplifyDemanded.cpp | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/llvm/lib/Transforms/InstCombine/InstCombineSimplifyDemanded.cpp b/llvm/lib/Transforms/InstCombine/InstCombineSimplifyDemanded.cpp index 31eb6aed31c..adc433c4b70 100644 --- a/llvm/lib/Transforms/InstCombine/InstCombineSimplifyDemanded.cpp +++ b/llvm/lib/Transforms/InstCombine/InstCombineSimplifyDemanded.cpp @@ -768,6 +768,7 @@ Value *InstCombiner::SimplifyDemandedUseBits(Value *V, APInt DemandedMask, // TODO: Could compute known zero/one bits based on the input. break; } + case Intrinsic::x86_mmx_pmovmskb: case Intrinsic::x86_sse_movmsk_ps: case Intrinsic::x86_sse2_movmsk_pd: case Intrinsic::x86_sse2_pmovmskb_128: @@ -776,9 +777,14 @@ Value *InstCombiner::SimplifyDemandedUseBits(Value *V, APInt DemandedMask, case Intrinsic::x86_avx2_pmovmskb: { // MOVMSK copies the vector elements' sign bits to the low bits // and zeros the high bits. - auto Arg = II->getArgOperand(0); - auto ArgType = cast<VectorType>(Arg->getType()); - unsigned ArgWidth = ArgType->getNumElements(); + unsigned ArgWidth; + if (II->getIntrinsicID() == Intrinsic::x86_mmx_pmovmskb) { + ArgWidth = 8; // Arg is x86_mmx, but treated as <8 x i8>. + } else { + auto Arg = II->getArgOperand(0); + auto ArgType = cast<VectorType>(Arg->getType()); + ArgWidth = ArgType->getNumElements(); + } // If we don't need any of low bits then return zero, // we know that DemandedMask is non-zero already. |