[clang] [llvm] [HLSL] Implement `WaveReadLaneAt` intrinsic (PR #111010)
Chris B via cfe-commits
cfe-commits at lists.llvm.org
Tue Oct 8 17:44:16 PDT 2024
================
@@ -87,6 +87,7 @@ class CGHLSLRuntime {
GENERATE_HLSL_INTRINSIC_FUNCTION(SDot, sdot)
GENERATE_HLSL_INTRINSIC_FUNCTION(UDot, udot)
GENERATE_HLSL_INTRINSIC_FUNCTION(WaveIsFirstLane, wave_is_first_lane)
+ GENERATE_HLSL_INTRINSIC_FUNCTION(WaveReadLaneAt, waveReadLaneAt)
----------------
llvm-beanz wrote:
This isn't just a convention for our backend, this is a convention common across LLVM backends. For example look at the AMD GCN intrinsics like `llvm.amdgcn.raw.ptr.buffer.store`.
I can see an argument that the `_`s were placed in the wrong places, but I think this was the wrong decision. A better name might be `wave_readlaneat`, but `camelCase` isn't really used in intrinsic names.
@bogner & @farzonl, we should discuss this before we break with common LLVM convention.
https://github.com/llvm/llvm-project/pull/111010
More information about the cfe-commits
mailing list