Commit e6c4c15
Jakub Chlanda
[LIBCLC] Fix mangling in atomic functions (#7057)
Hard coding the substitution for `AtomicFAddEXT` meant that for the case
without AS specified, it was pushing the `MemorySemanticFlag` inside
`Scope` namespace creating:
```cpp
__spirv_AtomicFAddEXT(double*, __spv::Scope::Flag, __spv::Scope::MemorySemanticsMask::Flag, double)
```
For the `Increment`, the number of chars for the mangled name `_Z24` was
correct, but the typo in the op name was resulting in the pointer being
lost, resulting in the first argument being just a value:
```cpp
__spirv_AtomicIncrementP(unsigned long AS1, __spv::Scope::Flag, __spv::Scope::MemorySemanticsMask::Flag)
```1 parent 20b1bea commit e6c4c15
File tree
2 files changed
+7
-7
lines changed- libclc/ptx-nvidiacl/libspirv/atomic
2 files changed
+7
-7
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
20 | 20 | | |
21 | 21 | | |
22 | 22 | | |
23 | | - | |
| 23 | + | |
24 | 24 | | |
25 | 25 | | |
26 | 26 | | |
| |||
30 | 30 | | |
31 | 31 | | |
32 | 32 | | |
33 | | - | |
| 33 | + | |
34 | 34 | | |
35 | 35 | | |
36 | 36 | | |
| |||
105 | 105 | | |
106 | 106 | | |
107 | 107 | | |
108 | | - | |
109 | | - | |
110 | | - | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
111 | 111 | | |
112 | 112 | | |
113 | 113 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
13 | | - | |
14 | | - | |
| 13 | + | |
| 14 | + | |
15 | 15 | | |
16 | 16 | | |
17 | 17 | | |
| |||
0 commit comments