Skip to content

Commit 1d8a6d6

Browse files
authored
[SYCL][NFC] Add a missing symbol (#6885)
While the copy assignment operator of AccessorImplHost is not used, it was auto generated by the compiler until recent changes and being checked in the ABI tests. The safest way to fix this is to just manually define it. Also removed 2 redundant lines in host_acc_opt test.
1 parent 9587467 commit 1d8a6d6

File tree

2 files changed

+12
-2
lines changed

2 files changed

+12
-2
lines changed

sycl/source/detail/accessor_impl.hpp

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,18 @@ class __SYCL_EXPORT AccessorImplHost {
5959
MElemSize(Other.MElemSize), MOffsetInBytes(Other.MOffsetInBytes),
6060
MIsSubBuffer(Other.MIsSubBuffer), MPropertyList(Other.MPropertyList) {}
6161

62+
AccessorImplHost &operator=(const AccessorImplHost &Other) {
63+
MAccData = Other.MAccData;
64+
MAccessMode = Other.MAccessMode;
65+
MSYCLMemObj = Other.MSYCLMemObj;
66+
MDims = Other.MDims;
67+
MElemSize = Other.MElemSize;
68+
MOffsetInBytes = Other.MOffsetInBytes;
69+
MIsSubBuffer = Other.MIsSubBuffer;
70+
MPropertyList = Other.MPropertyList;
71+
return *this;
72+
}
73+
6274
// The resize method provides a way to change the size of the
6375
// allocated memory and corresponding properties for the accessor.
6476
// These are normally fixed for the accessor, but this capability

sycl/test/basic_tests/accessor/host_acc_opt.cpp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,6 @@
99
// CHECK: define {{.*}}foo{{.*}} {
1010
// CHECK-NOT: call
1111
// CHECK-NOT: invoke
12-
// CHECK-NOT: call
13-
// CHECK-NOT: invoke
1412
// CHECK: load <4 x i32>
1513
// CHECK-NOT: call
1614
// CHECK-NOT: invoke

0 commit comments

Comments
 (0)