Commit 8fe0c4d
authored
[SYCL] Use SPIR-V built-in function call for all targets and add
Before this PR, SPIR-V built-in is represented as global variable for
SPIR/SPIR-V targets and as function call for other targets in
include/sycl/__spirv/spirv_vars.hpp.
According to
https://github.com/llvm/llvm-project/blob/main/llvm/docs/SPIRVUsage.rst,
SPIR-V built-in variable can be mapped to either function call or global
variable. So function call representation should work for SPIR-V target
as well.
The benefit of choosing function call over global variable is that
function call works better for AOT targets that bypass SPIR-V. It is
easier for these targets to implements SPIR-V built-in function via
built-in modules like libclc rather than special handling for lowering
global variable in a custom pass.
Add `BuiltIn` to the name to align with SPIR-V friendly IR and global
variable name.BuiltIn to the name (#19359)1 parent 7b2ac54 commit 8fe0c4d
File tree
92 files changed
+1251
-1819
lines changed- libclc/libspirv
- include/libspirv
- async
- workitem
- lib
- amdgcn-amdhsa
- assert
- group
- misc
- workitem
- amdgcn/workitem
- generic/workitem
- native_cpu/workitem
- ptx-nvidiacl
- group
- workitem
- r600/workitem
- libdevice
- include
- sanitizer
- llvm
- lib/SYCLLowerIR
- ESIMD
- test
- SYCLLowerIR
- SYCLOptimizeBarriers
- tools/sycl-post-link/sycl-esimd
- sycl
- include/sycl
- __spirv
- detail
- ext/oneapi
- experimental
- stl_wrappers
- test
- check_device_code
- esimd
- extensions/properties
- extensions
- include_deps
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
92 files changed
+1251
-1819
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
13 | | - | |
14 | | - | |
15 | | - | |
16 | | - | |
17 | | - | |
18 | | - | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
19 | 20 | | |
20 | 21 | | |
21 | 22 | | |
| |||
Lines changed: 1 addition & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
9 | | - | |
10 | | - | |
11 | | - | |
| 9 | + | |
Lines changed: 1 addition & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
9 | | - | |
10 | | - | |
11 | | - | |
| 9 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
9 | | - | |
10 | | - | |
11 | | - | |
| 9 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
9 | | - | |
10 | | - | |
11 | | - | |
| 9 | + | |
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
9 | | - | |
| 9 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
9 | | - | |
10 | | - | |
11 | | - | |
| 9 | + | |
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
9 | | - | |
| 9 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
9 | | - | |
10 | | - | |
11 | | - | |
| 9 | + | |
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
9 | | - | |
| 9 | + | |
0 commit comments