[clang] [llvm] [SPIRV] Add more id and range builtIns (PR #143909)

2025-06-12 Thread Victor Lomuller via cfe-commits
https://github.com/Naghasan ready_for_review https://github.com/llvm/llvm-project/pull/143909 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [SPIRV] Add more id and range builtIns (PR #143909)

2025-06-12 Thread Victor Lomuller via cfe-commits
https://github.com/Naghasan updated https://github.com/llvm/llvm-project/pull/143909 >From 049087b07503add00192211b8ee96bdfebcfeed3 Mon Sep 17 00:00:00 2001 From: Victor Lomuller Date: Thu, 5 Jun 2025 16:17:10 +0100 Subject: [PATCH] [SPIRV] Add more id and range builtIns The patch adds intrins

[clang] [llvm] [SPIRV] Add more id and range builtIns (PR #143909)

2025-06-12 Thread Victor Lomuller via cfe-commits
https://github.com/Naghasan converted_to_draft https://github.com/llvm/llvm-project/pull/143909 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [SPIRV] Add more id and range builtIns (PR #143909)

2025-06-12 Thread Victor Lomuller via cfe-commits
Naghasan wrote: The advantage of intrinsics is they can be understood better by passes, it also creates an easier way for passes to introduce them (and less error prone). It's kinda pedantic TBH and I don't have strong opinions about them (it just feels cleaner to me). For the frontend side, t

[clang] [llvm] [SPIRV] Add more id and range builtIns (PR #143909)

2025-06-12 Thread Nathan Gauër via cfe-commits
Keenuts wrote: Hi, question, why did you prefer passing an new intrinsic to the SPIR-V backend vs loading a global variable with the BuiltIn decoration? For example, when building `SV_Position` semantic in HLSL, we emit a global variable in the FE in the `Input` SC, along with the correct `spi

[clang] [llvm] [SPIRV] Add more id and range builtIns (PR #143909)

2025-06-12 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Victor Lomuller (Naghasan) Changes The patch adds intrinsics and lowering logic for GlobalSize, GlobalOffset, SubgroupMaxSize, NumWorkgroups, WorkgroupSize, WorkgroupId, LocalInvocationId, GlobalInvocationId, SubgroupSize, NumSubgroups, S

[clang] [llvm] [SPIRV] Add more id and range builtIns (PR #143909)

2025-06-12 Thread Victor Lomuller via cfe-commits
https://github.com/Naghasan created https://github.com/llvm/llvm-project/pull/143909 The patch adds intrinsics and lowering logic for GlobalSize, GlobalOffset, SubgroupMaxSize, NumWorkgroups, WorkgroupSize, WorkgroupId, LocalInvocationId, GlobalInvocationId, SubgroupSize, NumSubgroups, Subgrou