================
@@ -1115,7 +1115,7 @@ GCNTTIImpl::instCombineIntrinsic(InstCombiner &IC, 
IntrinsicInst &II) const {
   case Intrinsic::amdgcn_permlanex16_var: {
     // Discard vdst_in if it's not going to be read.
     Value *VDstIn = II.getArgOperand(0);
-    if (isa<UndefValue>(VDstIn))
+    if (isa<PoisonValue>(VDstIn))
----------------
nunoplopes wrote:

I don't know how prevalent this intrinsic is, but I would maybe leave this one 
as-is to preserve compatibility with older IR. We can always do a search & 
replace later when we get rid of undef.
The rest LGTM, thanks!

https://github.com/llvm/llvm-project/pull/131288
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to