Removal Of Duplicate Test Case from shift.ll file.

[sonamkumari]

Summary

Hi,

The removed duplicate test case matches with the following test case at Line No : 57.

Please help in reviewing it.

Regards,
Sonam.

[arsenm]

This shouldn't be removed, the duplicate test should be fixed. @test5_non_splat_vector at line 57 isn't supposed to be a duplicate.
at line 57 should be using a lshr not a shl

[sonamkumari]

Hi Matt,

Thanks for reviewing the patch. Made the changes as per your comments.
Please help in reviewing it.

Regards,
Sonam.

[arsenm]

LGTM

[sonamkumari]

Closed by commit rL223648 (authored by @sonamkumari )

test/Transforms/InstCombine/shift.ll

 ; CHECK-NEXT: ret <4 x i32> %A
   %B = lshr <4 x i32> %A, zeroinitializer
   ret <4 x i32> %B
 }
 
 define <4 x i32> @test5_non_splat_vector(<4 x i32> %A) {
 ; CHECK-LABEL: @test5_non_splat_vector(
 ; CHECK-NOT: ret <4 x i32> undef
-  %B = shl <4 x i32> %A, <i32 32, i32 1, i32 2, i32 3>
+  %B = lshr <4 x i32> %A, <i32 32, i32 1, i32 2, i32 3>
   ret <4 x i32> %B
 }
 
 define i32 @test5a(i32 %A) {
 ; CHECK-LABEL: @test5a(
 ; CHECK: ret i32 undef
         %B = shl i32 %A, 32     ;; shift all bits out
         ret i32 %B