[llvm-branch-commits] [llvm-branch] r214818 - Merging r214716:

[Bill Schmidt]

Author: wschmidt
Date: Mon Aug  4 18:41:27 2014
New Revision: 214818

URL: http://llvm.org/viewvc/llvm-project?rev=214818&view=rev
Log:
Merging r214716:
------------------------------------------------------------------------
r214716 | uweigand | 2014-08-04 08:27:12 -0500 (Mon, 04 Aug 2014) | 9 lines

[PowerPC] MULHU/MULHS are not legal for vector types

I ran into some test failures where common code changed vector division
by constant into a multiply-high operation (MULHU).  But these are not
implemented by the back-end, so we failed to recognize the insn.

Fixed by marking MULHU/MULHS as Expand for vector types.


------------------------------------------------------------------------

Added:
    llvm/branches/release_35/test/CodeGen/PowerPC/vec_urem_const.ll
Modified:
    llvm/branches/release_35/lib/Target/PowerPC/PPCISelLowering.cpp

Modified: llvm/branches/release_35/lib/Target/PowerPC/PPCISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/branches/release_35/lib/Target/PowerPC/PPCISelLowering.cpp?rev=214818&r1=214817&r2=214818&view=diff
==============================================================================
--- llvm/branches/release_35/lib/Target/PowerPC/PPCISelLowering.cpp (original)
+++ llvm/branches/release_35/lib/Target/PowerPC/PPCISelLowering.cpp Mon Aug  4 18:41:27 2014
@@ -453,6 +453,8 @@ PPCTargetLowering::PPCTargetLowering(PPC
       setOperationAction(ISD::EXTRACT_VECTOR_ELT, VT, Expand);
       setOperationAction(ISD::INSERT_VECTOR_ELT, VT, Expand);
       setOperationAction(ISD::BUILD_VECTOR, VT, Expand);
+      setOperationAction(ISD::MULHU, VT, Expand);
+      setOperationAction(ISD::MULHS, VT, Expand);
       setOperationAction(ISD::UMUL_LOHI, VT, Expand);
       setOperationAction(ISD::SMUL_LOHI, VT, Expand);
       setOperationAction(ISD::UDIVREM, VT, Expand);

Added: llvm/branches/release_35/test/CodeGen/PowerPC/vec_urem_const.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/branches/release_35/test/CodeGen/PowerPC/vec_urem_const.ll?rev=214818&view=auto
==============================================================================
--- llvm/branches/release_35/test/CodeGen/PowerPC/vec_urem_const.ll (added)
+++ llvm/branches/release_35/test/CodeGen/PowerPC/vec_urem_const.ll Mon Aug  4 18:41:27 2014
@@ -0,0 +1,10 @@
+; RUN: llc -mcpu=pwr6 -mattr=+altivec < %s
+
+; Common code used to replace the urem by a mulhu, and compilation would
+; then crash since mulhu isn't supported on vector types.
+
+define <4 x i32> @test(<4 x i32> %x) {
+entry:
+  %0 = urem <4 x i32> %x, <i32 2147483647, i32 2147483647, i32 2147483647, i32 2147483647>
+  ret <4 x i32> %0
+}