[PATCH] D82087: AMDGPU/clang: Add builtins for llvm.amdgcn.ballot

Tue Sep 20 11:43:59 PDT 2022

JonChesterfield added a comment.

In D82087#3803464 <https://reviews.llvm.org/D82087#3803464>, @arsenm wrote:

> In D82087#3797883 <https://reviews.llvm.org/D82087#3797883>, @jdoerfert wrote:
>
>> Can we land this? I'd like to use the new intrinsics as I don't understand the old ones.
>
> What do you think about using the two separate builtins, vs. one magic builtin that auto-changes the wavesize?

The magic one would also change its return type, or always be i64 with high bits (zext or maybe sext or maybe copy of low), so less magic seems clearer

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

https://reviews.llvm.org/D82087