[PATCH] D80422: Enable `align <n>` to be used in intrinsic definitions.

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D80422#2050474 <https://reviews.llvm.org/D80422#2050474>, @hliao wrote:

> To prepare the refactoring on D80364 <https://reviews.llvm.org/D80364>, intrinsics interested should be specified with the alignment on the return pointer. With this patch, `amdgcn.dispatch.ptr` is defined as follows
>
>   def int_amdgcn_dispatch_ptr :
>     Intrinsic<[LLVMQualPointerType<llvm_i8_ty, 4>], [],
>     [Align<-1, 4>, IntrNoMem, IntrSpeculatable]>;
>


Referring to the return index by a negative parameter index is a bit weird. How does this work for multiple return values? Is it [-2, -1]? This needs documenting somewhere


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80422