Skip to content

Commit 27cef14

Browse files
BensuoEwan Crawford
authored andcommitted
Update tag and address minor issues in new tests
1 parent f8b0dcb commit 27cef14

File tree

2 files changed

+11
-11
lines changed

2 files changed

+11
-11
lines changed

sycl/test-e2e/Graph/Inputs/local_accessor.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,11 @@ int main() {
2424
Queue.wait_and_throw();
2525

2626
auto node = add_node(Graph, Queue, [&](handler &CGH) {
27-
local_accessor<T, 1> localMem(LocalSize, CGH);
27+
local_accessor<T, 1> LocalMem(LocalSize, CGH);
2828

2929
CGH.parallel_for(nd_range({Size}, {LocalSize}), [=](nd_item<1> Item) {
30-
localMem[Item.get_local_linear_id()] = Item.get_global_linear_id() * 2;
31-
PtrA[Item.get_global_linear_id()] += localMem[Item.get_local_linear_id()];
30+
LocalMem[Item.get_local_linear_id()] = Item.get_global_linear_id() * 2;
31+
PtrA[Item.get_global_linear_id()] += LocalMem[Item.get_local_linear_id()];
3232
});
3333
});
3434

@@ -47,7 +47,7 @@ int main() {
4747

4848
for (size_t i = 0; i < Size; i++) {
4949
T Ref = 10 + i + (i * 2);
50-
(check_value(i, Ref, ReferenceA[i], "PtrA"));
50+
check_value(i, Ref, ReferenceA[i], "PtrA");
5151
}
5252

5353
return 0;

sycl/test-e2e/Graph/Inputs/whole_update_local_acc.cpp

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -9,12 +9,12 @@ auto add_graph_node(
99
exp_ext::command_graph<exp_ext::graph_state::modifiable> &Graph,
1010
queue &Queue, size_t Size, size_t LocalSize, T *Ptr) {
1111
return add_node(Graph, Queue, [&](handler &CGH) {
12-
local_accessor<T, 1> localMem(LocalSize, CGH);
12+
local_accessor<T, 1> LocalMem(LocalSize, CGH);
1313

1414
CGH.parallel_for(nd_range({Size}, {LocalSize}), [=](nd_item<1> Item) {
15-
localMem[Item.get_local_linear_id()] = Item.get_global_linear_id() * 2;
15+
LocalMem[Item.get_local_linear_id()] = Item.get_global_linear_id() * 2;
1616
Ptr[Item.get_global_linear_id()] +=
17-
localMem[Item.get_local_linear_id()] + Item.get_local_range(0);
17+
LocalMem[Item.get_local_linear_id()] + Item.get_local_range(0);
1818
});
1919
});
2020
}
@@ -61,8 +61,8 @@ int main() {
6161
for (size_t i = 0; i < Size; i++) {
6262
T RefA = 10 + i + (i * 2) + LocalSize / 2;
6363
T RefB = 10 + i;
64-
(check_value(i, RefA, ReferenceA[i], "PtrA"));
65-
(check_value(i, RefB, ReferenceB[i], "PtrB"));
64+
check_value(i, RefA, ReferenceA[i], "PtrA");
65+
check_value(i, RefB, ReferenceB[i], "PtrB");
6666
}
6767

6868
// Update GraphExecA using whole graph update
@@ -83,8 +83,8 @@ int main() {
8383
for (size_t i = 0; i < Size; i++) {
8484
T RefA = 10 + i + (i * 2) + LocalSize / 2;
8585
T RefB = 10 + i + (i * 2) + LocalSize;
86-
(check_value(i, RefA, ReferenceA[i], "PtrA"));
87-
(check_value(i, RefB, ReferenceB[i], "PtrB"));
86+
check_value(i, RefA, ReferenceA[i], "PtrA");
87+
check_value(i, RefB, ReferenceB[i], "PtrB");
8888
}
8989

9090
free(PtrA, Queue);

0 commit comments

Comments
 (0)