[PATCH] D80729: AMDGPU: Add IntrWillReturn to intrinsic definitions

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 28 10:59:39 PDT 2020


rampitec added inline comments.


================
Comment at: llvm/include/llvm/IR/IntrinsicsAMDGPU.td:188
    llvm_i32_ty],      // bit offset of the thread count
-  [IntrConvergent, ImmArg<ArgIndex<1>>]>;
+  [IntrConvergent, ImmArg<ArgIndex<1>>, IntrWillReturn]>;
 
----------------
arsenm wrote:
> rampitec wrote:
> > I am not sure this one always will return. With zero input it is a kill.
> I would expect a shader to never be launched with 0 input? I think it's also not clear if kill is really a return
I do not really know if someone may use it this way or not. Is it important to mark it?


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

https://reviews.llvm.org/D80729





More information about the llvm-commits mailing list