[PATCH] D127315: [AMDGPU] New GFX11 intrinsic llvm.amdgcn.s.sendmsg.rtn

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/include/llvm/IR/IntrinsicsAMDGPU.td:213
+def int_amdgcn_s_sendmsg_rtn : Intrinsic <[llvm_anyint_ty], [llvm_i32_ty],
+  [ImmArg<ArgIndex<0>>, IntrNoMem, IntrHasSideEffects]>;
+
----------------
Missing a lot of the default attributes (which all of the other intrinsics are also currently missing)


================
Comment at: llvm/test/CodeGen/AMDGPU/llvm.amdgcn.sendmsg.rtn.ll:127
+}
+
+declare i32 @llvm.amdgcn.s.sendmsg.rtn.i32(i32)
----------------
Maybe test some invalid immediates?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127315