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

[Pravin Jagtap via Phabricator via llvm-commits]

pravinjagtap added inline comments.


================
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,
----------------
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 ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156647