Apply suggestions

Author: haonanya1

llvm/lib/Transforms/InstCombine/InstCombinePHI.cpp

@@ -574,10 +574,8 @@ Instruction *InstCombinerImpl::foldPHIArgGEPIntoPHI(PHINode &PN) {
       // substantially cheaper to compute for the constants, so making it a
       // variable index could pessimize the path.  This also handles the case
       // for struct indices, which must always be constant.
-      if (isa<ConstantInt>(FirstInst->getOperand(Op)) ||
-          isa<ConstantInt>(GEP->getOperand(Op)) ||
-          isa<ConstantDataVector>(FirstInst->getOperand(Op)) ||
-          isa<ConstantDataVector>(GEP->getOperand(Op)))
+      if (isa<Constant>(FirstInst->getOperand(Op)) ||
+          isa<Constant>(GEP->getOperand(Op)))
         return nullptr;
 
       if (FirstInst->getOperand(Op)->getType() !=

llvm/test/Transforms/InstCombine/fold-phi-arg-gep-to-phi-negative.ll

@@ -4,10 +4,10 @@
 %vec_base = type { [4 x float] }
 %foo = type { %vec, %vec}
 
-define void @test(i1 %tobool, ptr addrspace(1) %add.ptr.i) {
+define void @test(i1 %tobool, ptr addrspace(1) %add.ptr) {
 entry:
   %lane.0 = alloca %foo, align 16
-  %lane.15 = insertelement <16 x ptr> undef, ptr %lane.0, i64 0
+  %lane.15 = insertelement <16 x ptr> poison, ptr %lane.0, i64 0
   %mm_vectorGEP = getelementptr inbounds %foo, <16 x ptr> %lane.15, <16 x i64> zeroinitializer, <16 x i32> splat (i32 1), <16 x i32> zeroinitializer, <16 x i32> zeroinitializer, <16 x i64> splat (i64 1)
   %mm_vectorGEP2 = getelementptr inbounds %foo, <16 x ptr> %lane.15, <16 x i64> zeroinitializer, <16 x i32> zeroinitializer, <16 x i32> zeroinitializer, <16 x i32> zeroinitializer, <16 x i64> splat (i64 1)
   br i1 %tobool, label %f1, label %f0
@@ -24,12 +24,8 @@ f1:
 
 merge:
 ; CHECK: merge:
-; CHECK-NEXT: %vec.phi14 = phi <16 x ptr> [ %mm_vectorGEP, %f0 ], [ %mm_vectorGEP2, %f1 ]
-  %vec.phi14 = phi <16 x ptr> [ %mm_vectorGEP, %f0], [ %mm_vectorGEP2, %f1 ]
-  %wide.masked.gather15 = call <16 x float> @llvm.masked.gather.v16f32.v16p0(<16 x ptr> %vec.phi14, i32 4, <16 x i1> splat (i1 true), <16 x float> poison)
-  %wide.masked.gather15.extract.15. = extractelement <16 x float> %wide.masked.gather15, i32 15
-  store float %wide.masked.gather15.extract.15., ptr addrspace(1) %add.ptr.i, align 4
+; CHECK-NEXT: %vec.phi = phi <16 x ptr> [ %mm_vectorGEP, %f0 ], [ %mm_vectorGEP2, %f1 ]
+  %vec.phi = phi <16 x ptr> [ %mm_vectorGEP, %f0], [ %mm_vectorGEP2, %f1 ]
+  store <16 x ptr> %vec.phi, ptr addrspace(1) %add.ptr
   ret void
 }
-
-declare <16 x float> @llvm.masked.gather.v16f32.v16p0(<16 x ptr>, i32 immarg, <16 x i1>, <16 x float>)