Skip to content

Commit 80e6c41

Browse files
authored
[SYCL][NFC] Remov unused arguments in addHostAccessor (#1830)
Signed-off-by: Stuart Adams <[email protected]>
1 parent 0098eab commit 80e6c41

File tree

3 files changed

+5
-7
lines changed

3 files changed

+5
-7
lines changed

sycl/source/detail/scheduler/graph_builder.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -393,8 +393,7 @@ Command *Scheduler::GraphBuilder::addCopyBack(Requirement *Req) {
393393

394394
// The function implements SYCL host accessor logic: host accessor
395395
// should provide access to the buffer in user space.
396-
Command *Scheduler::GraphBuilder::addHostAccessor(Requirement *Req,
397-
const bool destructor) {
396+
Command *Scheduler::GraphBuilder::addHostAccessor(Requirement *Req) {
398397

399398
const QueueImplPtr &HostQueue = getInstance().getDefaultHostQueue();
400399

sycl/source/detail/scheduler/scheduler.cpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -170,12 +170,11 @@ void Scheduler::removeMemoryObject(detail::SYCLMemObjI *MemObj) {
170170
MGraphBuilder.removeRecordForMemObj(MemObj);
171171
}
172172

173-
EventImplPtr Scheduler::addHostAccessor(Requirement *Req,
174-
const bool destructor) {
173+
EventImplPtr Scheduler::addHostAccessor(Requirement *Req) {
175174
std::unique_lock<std::shared_timed_mutex> Lock(MGraphLock, std::defer_lock);
176175
lockSharedTimedMutex(Lock);
177176

178-
Command *NewCmd = MGraphBuilder.addHostAccessor(Req, destructor);
177+
Command *NewCmd = MGraphBuilder.addHostAccessor(Req);
179178

180179
if (!NewCmd)
181180
return nullptr;

sycl/source/detail/scheduler/scheduler.hpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -410,7 +410,7 @@ class Scheduler {
410410
/// \param Req is the requirement to be updated.
411411
/// \return an event which indicates when these nodes are completed
412412
/// and host accessor is ready for use.
413-
EventImplPtr addHostAccessor(Requirement *Req, const bool Destructor = false);
413+
EventImplPtr addHostAccessor(Requirement *Req);
414414

415415
/// Unblocks operations with the memory object.
416416
///
@@ -472,7 +472,7 @@ class Scheduler {
472472
/// Enqueues a command to create a host accessor.
473473
///
474474
/// \param Req points to memory being accessed.
475-
Command *addHostAccessor(Requirement *Req, const bool destructor = false);
475+
Command *addHostAccessor(Requirement *Req);
476476

477477
/// [Provisional] Optimizes the whole graph.
478478
void optimize();

0 commit comments

Comments
 (0)