Index: llvm/trunk/lib/Transforms/InstCombine/InstCombineCalls.cpp =================================================================== --- llvm/trunk/lib/Transforms/InstCombine/InstCombineCalls.cpp +++ llvm/trunk/lib/Transforms/InstCombine/InstCombineCalls.cpp @@ -2085,6 +2085,10 @@ ConstantExpr::getSub(WidthC, ShAmtC)); } + // Left or right might be masked. + if (SimplifyDemandedInstructionBits(*II)) + return &CI; + // The shift amount (operand 2) of a funnel shift is modulo the bitwidth, // so only the low bits of the shift amount are demanded if the bitwidth is // a power-of-2. Index: llvm/trunk/test/Transforms/InstCombine/fsh.ll =================================================================== --- llvm/trunk/test/Transforms/InstCombine/fsh.ll +++ llvm/trunk/test/Transforms/InstCombine/fsh.ll @@ -561,8 +561,7 @@ define i32 @fshl_mask_args_same1(i32 %a) { ; CHECK-LABEL: @fshl_mask_args_same1( -; CHECK-NEXT: [[TMP1:%.*]] = and i32 [[A:%.*]], -65536 -; CHECK-NEXT: [[TMP2:%.*]] = call i32 @llvm.fshl.i32(i32 [[TMP1]], i32 [[TMP1]], i32 16) +; CHECK-NEXT: [[TMP2:%.*]] = lshr i32 [[A:%.*]], 16 ; CHECK-NEXT: ret i32 [[TMP2]] ; %tmp1 = and i32 %a, 4294901760 ; 0xffff0000 @@ -572,8 +571,8 @@ define i32 @fshl_mask_args_same2(i32 %a) { ; CHECK-LABEL: @fshl_mask_args_same2( -; CHECK-NEXT: [[TMP1:%.*]] = and i32 [[A:%.*]], 255 -; CHECK-NEXT: [[TMP2:%.*]] = call i32 @llvm.fshl.i32(i32 [[TMP1]], i32 [[TMP1]], i32 8) +; CHECK-NEXT: [[TMP1:%.*]] = shl i32 [[A:%.*]], 8 +; CHECK-NEXT: [[TMP2:%.*]] = and i32 [[TMP1]], 65280 ; CHECK-NEXT: ret i32 [[TMP2]] ; %tmp1 = and i32 %a, 255 @@ -583,8 +582,7 @@ define i32 @fshl_mask_args_same3(i32 %a) { ; CHECK-LABEL: @fshl_mask_args_same3( -; CHECK-NEXT: [[TMP1:%.*]] = and i32 [[A:%.*]], 255 -; CHECK-NEXT: [[TMP2:%.*]] = call i32 @llvm.fshl.i32(i32 [[TMP1]], i32 [[TMP1]], i32 24) +; CHECK-NEXT: [[TMP2:%.*]] = shl i32 [[A:%.*]], 24 ; CHECK-NEXT: ret i32 [[TMP2]] ; %tmp1 = and i32 %a, 255 @@ -594,9 +592,8 @@ define i32 @fshl_mask_args_different(i32 %a) { ; CHECK-LABEL: @fshl_mask_args_different( -; CHECK-NEXT: [[TMP2:%.*]] = and i32 [[A:%.*]], -65536 -; CHECK-NEXT: [[TMP1:%.*]] = and i32 [[A]], -16777216 -; CHECK-NEXT: [[TMP3:%.*]] = call i32 @llvm.fshl.i32(i32 [[TMP2]], i32 [[TMP1]], i32 17) +; CHECK-NEXT: [[TMP1:%.*]] = lshr i32 [[A:%.*]], 15 +; CHECK-NEXT: [[TMP3:%.*]] = and i32 [[TMP1]], 130560 ; CHECK-NEXT: ret i32 [[TMP3]] ; %tmp2 = and i32 %a, 4294901760 ; 0xfffff00f @@ -607,8 +604,7 @@ define <2 x i31> @fshr_mask_args_same_vector(<2 x i31> %a) { ; CHECK-LABEL: @fshr_mask_args_same_vector( -; CHECK-NEXT: [[TMP1:%.*]] = and <2 x i31> [[A:%.*]], -; CHECK-NEXT: [[TMP3:%.*]] = call <2 x i31> @llvm.fshl.v2i31(<2 x i31> [[A]], <2 x i31> [[TMP1]], <2 x i31> ) +; CHECK-NEXT: [[TMP3:%.*]] = shl <2 x i31> [[A:%.*]], ; CHECK-NEXT: ret <2 x i31> [[TMP3]] ; %tmp1 = and <2 x i31> %a, @@ -620,7 +616,7 @@ define <2 x i32> @fshr_mask_args_same_vector2(<2 x i32> %a, <2 x i32> %b) { ; CHECK-LABEL: @fshr_mask_args_same_vector2( ; CHECK-NEXT: [[TMP1:%.*]] = and <2 x i32> [[A:%.*]], -; CHECK-NEXT: [[TMP3:%.*]] = call <2 x i32> @llvm.fshl.v2i32(<2 x i32> [[TMP1]], <2 x i32> [[TMP1]], <2 x i32> ) +; CHECK-NEXT: [[TMP3:%.*]] = lshr exact <2 x i32> [[TMP1]], ; CHECK-NEXT: ret <2 x i32> [[TMP3]] ; %tmp1 = and <2 x i32> %a,