[PATCH] D151995: [AMDGPU] Add llvm.amdgcn.cs.chain intrinsic to IR & verifier

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/include/llvm/IR/IntrinsicsAMDGPU.td:1862
+  Intrinsic<[],
+            [llvm_ptr_ty, // The function to jump to.
+             llvm_anyint_ty, // Value to put in EXEC (should be i32 or i64).
----------------
Should probably use llvm_anyptr_ty


================
Comment at: llvm/include/llvm/IR/IntrinsicsAMDGPU.td:1872
+            ],
+            [IntrNoReturn, ImmArg<ArgIndex<4>>]>;
+
----------------
Should it be convergent, or leave that to callsites to set?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151995