[PATCH] D156647: [AMDGPU] Extend f32 support for llvm.amdgcn.update.dpp intrinsic

[Pravin Jagtap via Phabricator via llvm-commits]

pravinjagtap added a comment.

If there are no objections, I will go ahead and push this patch.



================
Comment at: llvm/include/llvm/IR/IntrinsicsAMDGPU.td:2180
 def int_amdgcn_update_dpp :
-  Intrinsic<[llvm_anyint_ty],
             [LLVMMatchType<0>, LLVMMatchType<0>, llvm_i32_ty,
----------------
pravinjagtap wrote:
> arsenm wrote:
> > this shouldn't break any existing clients, a name mangling param was needed anyway
> Are you referring to clients like LLPC ? Is there any way to make sure that its not breaking any of the clients ?
I ran LLPC pipeline, its clean. @foad 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D156647/new/

https://reviews.llvm.org/D156647