[clang] [AMDGPU] Use generic builtins for `wave_reduce` ops (PR #179589)

Alex Voicu via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 4 04:50:54 PST 2026


AlexVlx wrote:

> These aren't "generic builtins".
What do you actually mean here. Please use more words. The algorithm itself is generic, that it lowers to a dedicated intrinsic is inconsequential.

> 
> Also the FP and integer operations should absolutely not share a function name. Especially min and max

Why? Again, the algorithm itself is generic, you're just making life slightly harder for the user for no clear benefit, why would a HLL user care that the **intrinsic** that gets selected is reduce_fmin or reduce_fmax?

https://github.com/llvm/llvm-project/pull/179589


More information about the cfe-commits mailing list