[PATCH] D125378: [Attribute] Introduce shuffle attribute to be used for __shfl_sync like cross-lane APIs

krishna chaitanya sankisa via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 11 08:27:19 PDT 2022


skc7 added a comment.

In D125378#3506215 <https://reviews.llvm.org/D125378#3506215>, @jdoerfert wrote:

> In D125378#3506001 <https://reviews.llvm.org/D125378#3506001>, @nikic wrote:
>
>> Please specify the semantics of the new LLVM attribute in LangRef -- though I don't really understand why you need an LLVM-side attribute at all.
>
> +1. I doubt this patch is helpful.
>
>> Shuffle attribute has been added as per suggestions/comments from review: D124158 <https://reviews.llvm.org/D124158>
>
> I failed to see where this was suggested.
>
> ---
>
> My suggestion was and still is summarized in: https://reviews.llvm.org/D124158#3486110

@jdoerfert I meant to convey that, As a solution to review comments from D124158 <https://reviews.llvm.org/D124158>, introduced this shuffle attribute to identify __shfl_sync like APIs.


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

https://reviews.llvm.org/D125378



More information about the llvm-commits mailing list