[clang] [Headers] Implement spirvamdgcnintrin.h (PR #131164)

2025-03-18 Thread Jon Chesterfield via cfe-commits
JonChesterfield wrote: We don't need a way to call the builtins. See for example this pull request. Intel have done the spirv64-intel- thing similar to the spirv64-amd-amdhsa so they can (presumably?) use whatever intrinsics they like, using another header quite like this one. https://github.

[clang] [Headers] Implement spirvamdgcnintrin.h (PR #131164)

2025-03-18 Thread Joseph Huber via cfe-commits
https://github.com/jhuber6 commented: If all this does is call the `amdgpu` builtins, is it possible to just include that one? I think the only thing that differs is the address spaces so we could probably just undef those. https://github.com/llvm/llvm-project/pull/131164 _

[clang] [Headers] Implement spirvamdgcnintrin.h (PR #131164)

2025-03-18 Thread Jon Chesterfield via cfe-commits
https://github.com/JonChesterfield updated https://github.com/llvm/llvm-project/pull/131164 >From 402a091ac6eac8a50ce54a519acce5bfa4de1c88 Mon Sep 17 00:00:00 2001 From: Jon Chesterfield Date: Tue, 18 Mar 2025 15:57:02 + Subject: [PATCH] [Headers] Implement spirvamdgcnintrin.h --- clang/l

[clang] [Headers] Implement spirvamdgcnintrin.h (PR #131164)

2025-03-18 Thread Nick Sarnie via cfe-commits
sarnex wrote: > @sarnex I'm assuming you're somewhat familiar with SPIR-V and the backend. We > really need a way to call SPIR-V builtins directly, i.e. some way to get > clang to lower something like __builtin_spirv_ballot() to > OpGroupNonUniformBallot. I don't know how much SPIR-V exposes f

[clang] [Headers] Implement spirvamdgcnintrin.h (PR #131164)

2025-03-18 Thread Joseph Huber via cfe-commits
jhuber6 wrote: @sarnex I'm assuming you're somewhat familiar with SPIR-V and the backend. We really need a way to call SPIR-V builtins directly, i.e. some way to get clang to lower something like `__builtin_spirv_ballot()` to `OpGroupNonUniformBallot`. I don't know how much SPIR-V exposes for

[clang] [Headers] Implement spirvamdgcnintrin.h (PR #131164)

2025-03-18 Thread Nick Sarnie via cfe-commits
sarnex wrote: No problem, as long as we have a somewhat reasonable way to eventually get the DeviceRTL working I'm happy, thanks for working on this. https://github.com/llvm/llvm-project/pull/131164 ___ cfe-commits mailing list cfe-commits@lists.llvm.

[clang] [Headers] Implement spirvamdgcnintrin.h (PR #131164)

2025-03-18 Thread Jon Chesterfield via cfe-commits
https://github.com/JonChesterfield edited https://github.com/llvm/llvm-project/pull/131164 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Headers] Implement spirvamdgcnintrin.h (PR #131164)

2025-03-18 Thread Jon Chesterfield via cfe-commits
JonChesterfield wrote: @sarnex I'm deeply sorry about this sequence of events. The single spirv64 header that lowered to intrinsics that amdgpu or intel map onto their own world would have removed a swathe of spurious variation. What we're going to have to do in the interim is have spirvamdgcn

[clang] [Headers] Implement spirvamdgcnintrin.h (PR #131164)

2025-03-18 Thread Jon Chesterfield via cfe-commits
https://github.com/JonChesterfield edited https://github.com/llvm/llvm-project/pull/131164 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits