Commit d3cdb95
authored
[SYCL] Fix group algorithms for non-uniform groups, marray and vec (#14364)
This commit makes the following fixes to group algorithms:
* Some GroupNonUniform SPIR-V builtins were incorrectly named as they
were bundled together with their KHR Group SPIR-V equivalents. These
have been renamed to map correctly to the right SPIR-V operations.
* `sycl::marray` is now considered when checking for arithmetic types,
making it usable in group broadcast operations and the functions that
use them.
* The representation of `bool` in `sycl::vec` has been changed to
unsigned to match the representation picked by `ConvertToOpenCLType`.
* The representation of `char` in `sycl::vec` has been changed to to
match the representation picked by `ConvertToOpenCLType` to avoid cases
where signedness would cause type mismatches.
---------
Signed-off-by: Larsen, Steffen <[email protected]>1 parent 5f418ab commit d3cdb95
File tree
11 files changed
+452
-130
lines changed- sycl
- include
- CL/__spirv
- sycl
- detail
- ext/oneapi
- test-e2e/GroupAlgorithm
- test
- check_device_code/vector
- regression
11 files changed
+452
-130
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1117 | 1117 | | |
1118 | 1118 | | |
1119 | 1119 | | |
| 1120 | + | |
| 1121 | + | |
| 1122 | + | |
| 1123 | + | |
| 1124 | + | |
| 1125 | + | |
| 1126 | + | |
| 1127 | + | |
1120 | 1128 | | |
1121 | 1129 | | |
1122 | 1130 | | |
| |||
1182 | 1190 | | |
1183 | 1191 | | |
1184 | 1192 | | |
| 1193 | + | |
| 1194 | + | |
| 1195 | + | |
| 1196 | + | |
| 1197 | + | |
| 1198 | + | |
| 1199 | + | |
| 1200 | + | |
| 1201 | + | |
| 1202 | + | |
1185 | 1203 | | |
1186 | 1204 | | |
1187 | 1205 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
| 13 | + | |
13 | 14 | | |
14 | 15 | | |
15 | 16 | | |
16 | 17 | | |
17 | 18 | | |
18 | 19 | | |
19 | | - | |
20 | | - | |
21 | | - | |
22 | | - | |
23 | | - | |
24 | | - | |
25 | | - | |
26 | | - | |
27 | | - | |
28 | | - | |
29 | 20 | | |
30 | 21 | | |
31 | 22 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
236 | 236 | | |
237 | 237 | | |
238 | 238 | | |
239 | | - | |
240 | | - | |
| 239 | + | |
| 240 | + | |
241 | 241 | | |
242 | 242 | | |
243 | 243 | | |
| |||
747 | 747 | | |
748 | 748 | | |
749 | 749 | | |
750 | | - | |
| 750 | + | |
| 751 | + | |
751 | 752 | | |
752 | 753 | | |
753 | 754 | | |
754 | | - | |
755 | | - | |
| 755 | + | |
| 756 | + | |
| 757 | + | |
| 758 | + | |
756 | 759 | | |
757 | 760 | | |
758 | 761 | | |
| |||
761 | 764 | | |
762 | 765 | | |
763 | 766 | | |
764 | | - | |
765 | | - | |
| 767 | + | |
| 768 | + | |
766 | 769 | | |
767 | 770 | | |
768 | 771 | | |
| |||
780 | 783 | | |
781 | 784 | | |
782 | 785 | | |
783 | | - | |
| 786 | + | |
784 | 787 | | |
785 | 788 | | |
786 | 789 | | |
| |||
802 | 805 | | |
803 | 806 | | |
804 | 807 | | |
805 | | - | |
| 808 | + | |
806 | 809 | | |
807 | 810 | | |
808 | 811 | | |
| |||
1024 | 1027 | | |
1025 | 1028 | | |
1026 | 1029 | | |
1027 | | - | |
| 1030 | + | |
1028 | 1031 | | |
1029 | 1032 | | |
1030 | 1033 | | |
| |||
1033 | 1036 | | |
1034 | 1037 | | |
1035 | 1038 | | |
1036 | | - | |
| 1039 | + | |
1037 | 1040 | | |
1038 | 1041 | | |
1039 | 1042 | | |
| |||
1042 | 1045 | | |
1043 | 1046 | | |
1044 | 1047 | | |
1045 | | - | |
| 1048 | + | |
1046 | 1049 | | |
1047 | 1050 | | |
1048 | 1051 | | |
| |||
1051 | 1054 | | |
1052 | 1055 | | |
1053 | 1056 | | |
1054 | | - | |
| 1057 | + | |
1055 | 1058 | | |
1056 | 1059 | | |
1057 | 1060 | | |
| |||
1186 | 1189 | | |
1187 | 1190 | | |
1188 | 1191 | | |
1189 | | - | |
| 1192 | + | |
1190 | 1193 | | |
1191 | 1194 | | |
1192 | 1195 | | |
| |||
1201 | 1204 | | |
1202 | 1205 | | |
1203 | 1206 | | |
1204 | | - | |
1205 | | - | |
| 1207 | + | |
| 1208 | + | |
1206 | 1209 | | |
1207 | 1210 | | |
1208 | 1211 | | |
| |||
1286 | 1289 | | |
1287 | 1290 | | |
1288 | 1291 | | |
1289 | | - | |
1290 | | - | |
1291 | | - | |
| 1292 | + | |
| 1293 | + | |
| 1294 | + | |
1292 | 1295 | | |
1293 | | - | |
1294 | | - | |
1295 | | - | |
| 1296 | + | |
| 1297 | + | |
| 1298 | + | |
1296 | 1299 | | |
1297 | | - | |
1298 | | - | |
| 1300 | + | |
| 1301 | + | |
1299 | 1302 | | |
1300 | | - | |
1301 | | - | |
1302 | | - | |
| 1303 | + | |
| 1304 | + | |
| 1305 | + | |
1303 | 1306 | | |
1304 | | - | |
1305 | | - | |
1306 | | - | |
| 1307 | + | |
| 1308 | + | |
| 1309 | + | |
1307 | 1310 | | |
1308 | | - | |
1309 | | - | |
| 1311 | + | |
| 1312 | + | |
1310 | 1313 | | |
1311 | 1314 | | |
1312 | 1315 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
156 | 156 | | |
157 | 157 | | |
158 | 158 | | |
159 | | - | |
| 159 | + | |
160 | 160 | | |
161 | 161 | | |
162 | | - | |
| 162 | + | |
163 | 163 | | |
164 | 164 | | |
165 | | - | |
| 165 | + | |
166 | 166 | | |
167 | 167 | | |
168 | 168 | | |
169 | 169 | | |
170 | | - | |
| 170 | + | |
171 | 171 | | |
172 | | - | |
| 172 | + | |
173 | 173 | | |
174 | 174 | | |
175 | 175 | | |
176 | 176 | | |
177 | | - | |
| 177 | + | |
178 | 178 | | |
179 | 179 | | |
180 | 180 | | |
181 | | - | |
| 181 | + | |
182 | 182 | | |
183 | 183 | | |
184 | 184 | | |
| |||
194 | 194 | | |
195 | 195 | | |
196 | 196 | | |
197 | | - | |
| 197 | + | |
198 | 198 | | |
199 | 199 | | |
200 | 200 | | |
| 201 | + | |
| 202 | + | |
| 203 | + | |
201 | 204 | | |
202 | 205 | | |
203 | 206 | | |
| |||
295 | 298 | | |
296 | 299 | | |
297 | 300 | | |
| 301 | + | |
| 302 | + | |
| 303 | + | |
| 304 | + | |
| 305 | + | |
| 306 | + | |
| 307 | + | |
| 308 | + | |
| 309 | + | |
| 310 | + | |
| 311 | + | |
| 312 | + | |
| 313 | + | |
| 314 | + | |
298 | 315 | | |
299 | 316 | | |
300 | 317 | | |
| |||
304 | 321 | | |
305 | 322 | | |
306 | 323 | | |
307 | | - | |
| 324 | + | |
308 | 325 | | |
309 | 326 | | |
310 | 327 | | |
| |||
314 | 331 | | |
315 | 332 | | |
316 | 333 | | |
317 | | - | |
| 334 | + | |
318 | 335 | | |
319 | 336 | | |
320 | 337 | | |
| |||
324 | 341 | | |
325 | 342 | | |
326 | 343 | | |
327 | | - | |
| 344 | + | |
| 345 | + | |
328 | 346 | | |
329 | 347 | | |
330 | 348 | | |
331 | 349 | | |
332 | 350 | | |
333 | | - | |
334 | | - | |
| 351 | + | |
| 352 | + | |
| 353 | + | |
| 354 | + | |
335 | 355 | | |
336 | 356 | | |
337 | 357 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
79 | 79 | | |
80 | 80 | | |
81 | 81 | | |
82 | | - | |
83 | | - | |
84 | | - | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
85 | 85 | | |
86 | 86 | | |
87 | | - | |
88 | | - | |
89 | | - | |
90 | | - | |
91 | | - | |
92 | | - | |
93 | | - | |
94 | | - | |
95 | | - | |
96 | | - | |
97 | | - | |
98 | | - | |
99 | | - | |
100 | | - | |
101 | | - | |
102 | | - | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
103 | 103 | | |
104 | 104 | | |
105 | 105 | | |
| |||
0 commit comments